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

Add route to view all orders #171

Closed
wants to merge 1 commit into from
Closed

Add route to view all orders #171

wants to merge 1 commit into from

Conversation

Jan-PieterBaert
Copy link
Member

This implements and closes #126

@redfast00
Copy link
Member

@Jan-PieterBaert @SchrodingersCat00 What's the usecase of this feature? This will be a pretty heavy page to load, and will not be in a useful format (HTML instead of something like JSON).

@wschella
Copy link
Member

@redfast00 sometimes you need look for past orders to fix some personal financing

@Jan-PieterBaert Jan-PieterBaert requested a review from mcbloch April 19, 2022 18:42
@mcbloch
Copy link
Member

mcbloch commented Apr 19, 2022

We can support this for admins or for showing your own orders. We do not want to publicly list all orders as we are even working on anonymizing all of them with unique links.

@Jan-PieterBaert
Copy link
Member Author

We can support this for admins or for showing your own orders. We do not want to publicly list all orders as we are even working on anonymizing all of them with unique links.

In that case I'll close this issue and you might consider changing #126 as well

@mcbloch
Copy link
Member

mcbloch commented Apr 19, 2022

In that case I'll close this issue and you might consider changing #126 as well

Good catch

@mcbloch mcbloch deleted the feature-126 branch April 19, 2022 20:53
@mcbloch mcbloch restored the feature-126 branch April 20, 2022 15:12
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.

Create order history page
4 participants