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

Changed a single line in turbo to insert cloned products after the original product #12675

Conversation

zsoltiNagy
Copy link
Contributor

Changed a single line in turbo to insert cloned products after the original product

This small change is needed to address the issue brought up in #12623

What should we test?

Only the visual behavior.

  • feature toggle the admin_style_v3

  • Visit /admin/products

  • Clone a product, see if it appears below or above the original, it should appear below it

  • User facing changes

  • API changes (V0, V1, DFC or Webhook)

  • Technical changes only

  • Feature toggled

Dependencies

none

Documentation updates

none

Copy link
Member

@dacook dacook left a comment

Choose a reason for hiding this comment

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

Hooray, thank you!

@dacook dacook added the feature toggled These pull requests' changes are invisible by default and are grouped in release notes label Jul 16, 2024
Copy link
Collaborator

@chahmedejaz chahmedejaz left a comment

Choose a reason for hiding this comment

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

That's great! Thank you so much 🎉

@filipefurtad0 filipefurtad0 self-assigned this Jul 17, 2024
@filipefurtad0 filipefurtad0 added the pr-staged-au staging.openfoodnetwork.org.au label Jul 17, 2024
@filipefurtad0
Copy link
Contributor

Hey @zsoltiNagy , I believe this is your first contribution, right? Welcome! 👋

Before this PR, cloning a product inserts the clone above the original (please ignore the save changes banner, in the pic below, it's not relevant here):

image

After this PR, reloading the page, displays the clone below the original:

image

And creating another clone correctly adds the new product beneath it:

image

This looks great! Merging 🎉

@filipefurtad0 filipefurtad0 merged commit f3e7ba0 into openfoodfoundation:master Jul 17, 2024
55 checks passed
@filipefurtad0 filipefurtad0 removed the pr-staged-au staging.openfoodnetwork.org.au label Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature toggled These pull requests' changes are invisible by default and are grouped in release notes
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[BUU] Insert cloned products below the original product
4 participants