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

New order insight popup bug #134

Open
triszt4n opened this issue Mar 7, 2022 · 2 comments · May be fixed by #135
Open

New order insight popup bug #134

triszt4n opened this issue Mar 7, 2022 · 2 comments · May be fixed by #135
Assignees

Comments

@triszt4n
Copy link
Member

triszt4n commented Mar 7, 2022

Bug catched in

PRODUCTION

Bug report

As I ordered an Americano chicken burger:

image

I've chosen 'Pirított hagyma' as an extra. But when I checked in the profile page, in the order's insight popup, the extra was called 'Tükörtojás':

image

The issue might have come up because on all the other burgers the third extra is called 'Tükörtojás':

image

Other request

Someone with PDF generation rights should try out the PDF generation if the extra is well named in the PDF (just so that the Americano guys won't switch up every 'Pirított hagyma' request for 'Tükörtojás').

@Gerviba
Copy link
Member

Gerviba commented Mar 7, 2022

image
The pdf generation works as intended. The issue is related to the new info popup details generation.

Thanks for the report!

@nemaberci nemaberci self-assigned this Mar 8, 2022
nemaberci pushed a commit that referenced this issue Mar 8, 2022
@nemaberci nemaberci linked a pull request Mar 8, 2022 that will close this issue
nemaberci pushed a commit that referenced this issue Mar 16, 2022
@weiszpal
Copy link

weiszpal commented May 15, 2023

I'm still having the same issue, moreover it was shown incorrectly in the price calculation tab, but the final price remained the same as 'Alapár'.
image
image

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 a pull request may close this issue.

4 participants