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

Change the link for "Enable filtering" #2187

Merged
merged 4 commits into from
Jan 5, 2023
Merged

Change the link for "Enable filtering" #2187

merged 4 commits into from
Jan 5, 2023

Conversation

jas7457
Copy link
Contributor

@jas7457 jas7457 commented Dec 23, 2022

PR Summary:

Filters are now edited in the Search & Discovery app. This changes the link so the user can learn how to customize filters via the app.

Why are these changes introduced?

Fixes https://github.com/Shopify/discovery-app/issues/1098

What approach did you take?

Copied/pasted text :D

Other considerations

Decision log

# Decision Alternatives Rationale Downsides
1

Visual impact on existing themes

They'll set an updated link: Video

Demo links

Checklist

@jas7457 jas7457 requested a review from NathanPJF December 23, 2022 15:19
NathanPJF
NathanPJF previously approved these changes Dec 23, 2022
Copy link
Contributor

@NathanPJF NathanPJF left a comment

Choose a reason for hiding this comment

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

FYI for other reviewers. This content is using a similar structure that we saw here when talking about review apps.

Which is shown in the same panel as this code change.

@andrewetchen andrewetchen self-requested a review December 23, 2022 16:34
@jas7457
Copy link
Contributor Author

jas7457 commented Jan 3, 2023

Looks like I need two approving reviews on this, going to assign to some more people as well.

Copy link
Contributor

@FCalabria FCalabria left a comment

Choose a reason for hiding this comment

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

LGTM

@jas7457 jas7457 merged commit 3e6bc97 into main Jan 5, 2023
@jas7457 jas7457 deleted the disco-link branch January 5, 2023 13:25
phapsidesGT pushed a commit to Gravytrain-UK/gt-shopify-dawn-theme that referenced this pull request Sep 3, 2024
* Change the link for "Enable filtering"

* Update 15 translation files

* Update 4 translation files

* Update 1 translation file

Co-authored-by: translation-platform[bot] <34770790+translation-platform[bot]@users.noreply.github.com>
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.

3 participants