Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modify info string for gift card recipient checkbox #2588

Merged
merged 4 commits into from
May 2, 2023

Conversation

mzarud
Copy link
Contributor

@mzarud mzarud commented Apr 28, 2023

PR Summary:

Closes https://github.com/Shopify/core-issues/issues/54765

This PR changes an informational string in the theme editor that is found beside the gift card recipient form enable/disable checkbox.

Existing string:
image

Why are these changes introduced?

We are releasing scheduling functionality in the recipient form with the next theme release, so we want to acknowledge that in the string.

Checklist

@mzarud mzarud requested a review from ludoboludo April 28, 2023 15:15
@mzarud mzarud marked this pull request as ready for review April 28, 2023 15:15
Copy link
Contributor

@ludoboludo ludoboludo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good, ill just ping a designer to confirm they're good with it too :)

@katycobb
Copy link

katycobb commented May 1, 2023

Hey @ludoboludo and @mzarud ! @danielvan asked me to take a look at the copy here. I have a suggestion for making it a bit shorter/clearer:

Checkbox: Show recipient information form for gift cards
Help text: Allows buyers to send gift cards on a scheduled date along with a personal message. Learn more

lmk if you have any feedback - I realize I'm popping in without a ton of context so definitely open to your feedback.

@@ -2008,7 +2008,7 @@
},
"show_gift_card_recipient": {
"label": "Show recipient information form for gift card products",
"info": "Gift card products can optionally be sent direct to a recipient along with a personal message. [Learn more](https://help.shopify.com/manual/online-store/themes/customizing-themes/add-gift-card-recipient-fields)"
"info": "Gift card products can optionally be sent direct to a recipient on a scheduled date along with a personal message. [Learn more](https://help.shopify.com/manual/online-store/themes/customizing-themes/add-gift-card-recipient-fields)"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might be a little too late to change for the potential release this week. So maybe we can make that change you suggested @katycobb if @mzarud's team is good with it as well in a follow up for the next release 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That works for us. I think we have some other minor improvements to ship in the next release anyways so I've created a GH issue to track this as well.

@ludoboludo
Copy link
Contributor

I asked if we could rush the translations again FYI 😅 We should have them all soon 👍

@ludoboludo ludoboludo merged commit d5274c8 into main May 2, 2023
@ludoboludo ludoboludo deleted the mzarud/modify_gc_string branch May 2, 2023 20:13
jabrms pushed a commit to fp-trading/fp-trading-theme that referenced this pull request Jun 6, 2023
jabrms pushed a commit to fp-trading/fp-trading-theme that referenced this pull request Jun 12, 2023
pangloss added a commit to pangloss/dawn that referenced this pull request Jun 14, 2023
* shopify/main: (59 commits)
  [Announcement bar] Add social icons (Shopify#2497)
  Update theme version to match the pubic release (Shopify#2698)
  Add release/v10.0.0 branch fixes to main (Shopify#2693)
  fix default UI for dropdown and mega menu (Shopify#2644)
  Fix link formatting in Related Products heading (Shopify#2680)
  Update 2 translation files (Shopify#2637)
  Enable gift card recipient form by default on featured product section (Shopify#2666)
  Gift cards/enable recipient form by default (Shopify#2618)
  Add a Color Scheme setting for Menus-Header (Shopify#2622)
  Made mobile drawer full width by default-header (Shopify#2625)
  Allow multiple announcement bars in Header group (Shopify#2619)
  [Feat Product] Add rating styling sheet (Shopify#2620)
  Fix password page variables (Shopify#2607)
  Fix transform applied when it should not for sliders (Shopify#2606)
  Modify info string for gift card recipient checkbox (Shopify#2588)
  [3D lift animation] Raise hovered card above adjacent cards (Shopify#2589)
  Remove fallback color scheme info text (Shopify#2602)
  Fix CSS specifity issue to cancel animation for theme editor events (Shopify#2605)
  Remove settings daya for icon color (Shopify#2601)
  Follow ups for accessibility of announcements slider (Shopify#2580)
  ...
phapsidesGT pushed a commit to Gravytrain-UK/gt-shopify-dawn-theme that referenced this pull request Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants