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

[Vouchers] The note regarding order total less than voucher amount shouldn't be displayed for percentage vouchers #11666

Closed
drummer83 opened this issue Aug 8, 2023 · 9 comments · Fixed by #11694
Assignees
Labels
bug-s4 The bug is annoying, but doesn't prevent from using the platform. Not so many users are impacted. good first issue hackathon Issues for upcoming hackathons papercut Labels papercuts after they've moved from the "Papercuts prioritized" column

Comments

@drummer83
Copy link
Contributor

drummer83 commented Aug 8, 2023

Description

The note 'Note: if your order total is less than your voucher you may not be able to spend the remaining value.' displayed at step 2 of checkout shouldn't be displayed for percentage vouchers because in this case it can't happen.

image

Steps to Reproduce

  1. As an enterprise manager, Set up a percentage voucher on a shop (make sure the feature toggle "vouchers" is activated"
  2. As a shopper, apply that voucher to step 2 of the checkout process - notice the message is not displayed
  3. Retest with a non percentage voucher and check the message is showing up

Let's add an automated test for this

Severity

bug-s4: it's annoying, but you can use it

@lin-d-hop
Copy link
Contributor

Confirmed papercut :)

@RachL RachL transferred this issue from openfoodfoundation/wishlist Oct 13, 2023
@RachL RachL added hackathon Issues for upcoming hackathons good first issue bug-s4 The bug is annoying, but doesn't prevent from using the platform. Not so many users are impacted. papercut Labels papercuts after they've moved from the "Papercuts prioritized" column labels Oct 13, 2023
@prateek0411999
Copy link
Contributor

Hey @drummer83 - I'd like to work on this issue. Can I please get assigned to it? Thanks :)

@drummer83
Copy link
Contributor Author

Of course, and welcome, @prateek0411999!
Looking forward to your contribution!
And don't hesitate to ping us if you get stuck.

@prateek0411999
Copy link
Contributor

Hi @drummer83, I've raised a pull request for the same (#11694)

Let me know if any change is required in this

@drummer83
Copy link
Contributor Author

Thank you, @prateek0411999!
Your pull request will now be reviewed by our developers and then tested. We will let you know!
Thanks again!

@rioug
Copy link
Collaborator

rioug commented Oct 23, 2023

Hey @drummer83 , as far as I can see the problem you are describing shouldn't be happening right now. I have not been able to reproduce it, could you check again ? and if you manage to reproduce it, could you provide more detailed step to reproduce ?

I have a feeling this might have been an issue at some point, but it got resolved as part of another bug fix.

@prateek0411999
Copy link
Contributor

prateek0411999 commented Oct 23, 2023

Hey @rioug, As per my observation this issue is still persisted, i was able to reproduce the same as well that's why i've raised the PR.

@filipefurtad0
Copy link
Contributor

filipefurtad0 commented Oct 26, 2023

Agree, this one is not clear how to reproduce... Below a percentage voucher is added, but no warning is displayed. This is before staging the associated PR:

image

Also the pic in the description displays a flat amount voucher. Any ideas?

Edit: got it now - order amount needs to be lower that the voucher percentage value.

@filipefurtad0 filipefurtad0 added no-staging-UK A tag which does not trigger deployments, indicating a server is being used and removed no-staging-UK A tag which does not trigger deployments, indicating a server is being used labels Oct 26, 2023
@filipefurtad0
Copy link
Contributor

Thanks @prateek0411999 - your spec documented it nicely 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-s4 The bug is annoying, but doesn't prevent from using the platform. Not so many users are impacted. good first issue hackathon Issues for upcoming hackathons papercut Labels papercuts after they've moved from the "Papercuts prioritized" column
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

6 participants