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
feat(backend): added orders crud + Operations creation(including automatic wallet debit+credit) #58
Conversation
- update Orders - add OrdersArticles
…y providing theirs id
- update GetOrganisationOrdersHandler - add CreateOrderHandler
- CreateOrder - DeclineOrder - AcceptOrder - PayOrder
- DeleteOrderHandler - ProcessOrderHandler - PayOrderHandler
❌ Deploy Preview for reverent-hawking-9ced65 failed. 🔨 Explore the source changes: c1fed38 🔍 Inspect the deploy log: https://app.netlify.com/sites/reverent-hawking-9ced65/deploys/612d650b33ff86000773d1ce |
Co-authored-by: Boris Mbarga <elhmn@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for this PR !
- Things looks good to me
- Make sure to get latest modifications from the master branch/and fix conflicts
- This PR is for so many subject that for me it was a little bit hard to follow things well, please, if possible in the future, make sure to manage PR one subject at time !
OperationType: string(operationType), | ||
Approved: true, | ||
//due to nullity constraint in migrations we give an initial random value to our operation hash | ||
OperationHash: fmt.Sprintf("%v", time.Now().UTC()), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏾
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for this PR !
* Things looks good to me * Make sure to get latest modifications from the master branch/and fix conflicts * This PR is for so many subject that for me it was a little bit hard to follow things well, please, if possible in the future, make sure to manage PR one subject at time !
Thanks. Next time i will manage PR one subject at time
# Conflicts: # backend/go.mod # backend/go.sum
Fantastic job @DipandaAser I am merging this 🥳 |
This pull request managing orders
Why ?
User need to order items for spent YOTAS 🤑. Admin need to manage those orders by accepting or declining
How ?
Steps to verify:
** Screenshots (optional)