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

Fix mapping of flash messages on order cycle pages #11885

Merged
merged 3 commits into from Dec 6, 2023

Conversation

dacook
Copy link
Member

@dacook dacook commented Dec 1, 2023

What? Why?

While working on flash messages I noticed these could be tidied up.

I think lazy-lookup is the way to go for helping organise the locale file. However it does mean these strings might need re-translating. Does Transifex automatically copy duplicates for you? We shall see..

What should we test?

Test on the old design, or if #11874 has been merged, you can check the new admin_style_v3 design.
The success messages now appear as a success, not a 'notice'. (Green in the old design, black in the new)

  • Order cycle create
  • Order cycle update
  • Order cycle clone
  • Order cycle notify producers

Release notes

Changelog Category (reviewers may add a label for the release notes):

  • User facing changes
  • API changes (V0, V1, DFC or Webhook)
  • Technical changes only
  • Feature toggled

The title of the pull request will be included in the release notes.

Dependencies

Documentation updates

@dacook dacook self-assigned this Dec 1, 2023
Copy link
Collaborator

@rioug rioug left a comment

Choose a reason for hiding this comment

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

Nice one 👍

Copy link
Member

@mkllnk mkllnk left a comment

Choose a reason for hiding this comment

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

Great!

@drummer83 drummer83 self-assigned this Dec 6, 2023
@drummer83 drummer83 added no-staging-FR A tag which does not trigger deployments, indicating a server is being used pr-staged-fr staging.coopcircuits.fr and removed no-staging-FR A tag which does not trigger deployments, indicating a server is being used labels Dec 6, 2023
@drummer83
Copy link
Contributor

Hi @dacook,
That's a nice improvement. Thanks! 👍

Test results

Action Before After
Create grafik grafik
Update (Save and Next) grafik grafik
Clone grafik grafik
Notify producers no confirmation no confirmation

Notes

  • Flash message is only shown when clicking Save and Next.
  • Clicking Save only shows an in-line message:
    grafik

Issues out of scope

  • No confirmation when clicking Save or Save and Back to List on step 4 'Checkout options'.

Conclusion

This fixed the incorrect mapping of flash messages on the order cycle pages.
No regressions were found. Merging! 🎉

Thanks again! 🤗

@drummer83 drummer83 changed the title Flash messages Fix mapping of flash messages on order cycle pages Dec 6, 2023
@drummer83 drummer83 merged commit dc4b651 into openfoodfoundation:master Dec 6, 2023
54 checks passed
@drummer83 drummer83 removed the pr-staged-fr staging.coopcircuits.fr label Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

4 participants