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.

Sign up for GitHub

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

Jump to bottom

Update success / notice / error messages with the correct status #11884

Open
mariocarabotta opened this issue Dec 1, 2023 · 2 comments
Open

Update success / notice / error messages with the correct status #11884

mariocarabotta opened this issue Dec 1, 2023 · 2 comments
Labels
design-needed issues requiring design
Milestone
[BUU2] Product Li...

Comments

@mariocarabotta
Copy link
Collaborator

mariocarabotta commented Dec 1, 2023 •
edited

to be mapped first. below an example of a message that should be success and not notice.
more info https://openfoodnetwork.slack.com/archives/C01CXQNJ1J6/p1701325935851049

Screenshot 2023-12-01 at 13 49 38

Screenshot 2023-12-19 at 13 23 26

the above happens also when updating a customer details
This was referenced Dec 1, 2023
Closed
Merged
@dacook dacook mentioned this issue Dec 1, 2023
Merged
4 tasks
@mariocarabotta mariocarabotta added the design-needed issues requiring design label Dec 5, 2023
@drummer83 drummer83 mentioned this issue Dec 22, 2023
Closed
4 tasks
@RachL
Copy link
Contributor

RachL commented Apr 8, 2024

@mariocarabotta will you have time to design something here or shall we skip this one as part of the first release?

@mariocarabotta
Copy link
Collaborator Author

skip. There's no real design involved, it's more about mapping messages that have the wrong status. The order cycle creation has been fixed already.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
design-needed issues requiring design
Projects
OFN Delivery board
Status: All the things
Milestone
[BUU2] Product List uplift
Development

No branches or pull requests
2 participants
@RachL @mariocarabotta