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(payment-option): hide dibs when no previous orders #697

Merged
merged 1 commit into from
Aug 29, 2023

Conversation

AdrianAndersen
Copy link
Member

Currently, handout orders do not produce a movedFromOrder, and thus
the originalOrderIds array will be empty. Since we just check that every
order is by the customer, this will return true if there are no orders.
Therefore, we have to reject DIBS if there are no previous orders.

Currently, handout orders do _not_ produce a movedFromOrder, and thus
the originalOrderIds array will be empty. Since we just check that every
order is by the customer, this will return true if there are no orders.
Therefore, we have to reject DIBS if there are no previous orders.
@vercel
Copy link

vercel bot commented Aug 25, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
bl-admin ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 25, 2023 8:36am

@AdrianAndersen
Copy link
Member Author

Copy link
Member

@LarsSelbekk LarsSelbekk left a comment

Choose a reason for hiding this comment

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

This looks good, but shouldn't we prevent it on the API side as well? 🤠

@AdrianAndersen
Copy link
Member Author

This looks good, but shouldn't we prevent it on the API side as well? 🤠

Neh, this is just for accounting

@AdrianAndersen AdrianAndersen merged commit ca21429 into master Aug 29, 2023
4 checks passed
@AdrianAndersen AdrianAndersen deleted the hide-dibs-option branch August 29, 2023 09:19
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 this pull request may close these issues.

2 participants