goc_daniel/b2b
2
0
Fork 0
Code Issues 8 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

orders #58

Merged
goc_daniel merged 13 commits from orders into main 2026-04-14 11:20:06 +00:00
Conversation 3 Commits 13 Files Changed 24 +750 -61
goc_daniel commented 2026-04-10 12:58:06 +00:00
Owner
Copy Link

Basic orders mechanics.

Please check if the permissions look good.

Basic orders mechanics. Please check if the permissions look good.
goc_daniel added 9 commits 2026-04-10 12:58:07 +00:00
orders tables 438a13c04c
orders handler init dfdf8b4db9
order struct 4f4b32b131
basic setup a6aa06faa0
Find is done 8595969c6e
code ready, time for testing... 134bc4ea53
small fix a03a2b461f
basic orders are ready 33e9d016e9
GET -> POST 80d26bba12
goc_daniel added 1 commit 2026-04-10 13:17:33 +00:00
send email when creating new order d4d55e2757
goc_daniel requested review from dudzic_wiktor 2026-04-10 13:17:53 +00:00
dudzic_wiktor reviewed 2026-04-10 13:45:43 +00:00
app/service/orderService/orderService.go Outdated
@@ -0,0 +87,4 @@
if err != nil {
return err
}
dudzic_wiktor commented 2026-04-10 13:44:54 +00:00
Collaborator
Copy Link

after successful order placement we should either delete or deactivate the cart that is leftover

after successful order placement we should either delete or deactivate the cart that is leftover
goc_daniel marked this conversation as resolved
app/service/orderService/orderService.go Outdated
@@ -0,0 +89,4 @@
}
// send email to admin
err = s.emailService.SendNewOrderPlacedNotification(user_id)
dudzic_wiktor commented 2026-04-10 13:36:18 +00:00
Collaborator
Copy Link

should be a goroutine to not stall the user from getting response from the server

should be a goroutine to not stall the user from getting response from the server
goc_daniel marked this conversation as resolved
app/service/orderService/orderService.go
@@ -0,0 +119,4 @@
}
func (s *OrderService) ChangeOrderStatus(user *model.Customer, order_id uint, status string) error {
if !user.HasPermission(perms.ModifyAllOrders) {
dudzic_wiktor commented 2026-04-10 13:40:40 +00:00
Collaborator
Copy Link

standard user never changes it's own order status, only admin has privilege to do that. endpoint should forbid access at the handler level

standard user never changes it's own order status, only admin has privilege to do that. endpoint should forbid access at the handler level
goc_daniel marked this conversation as resolved
goc_daniel added 1 commit 2026-04-13 07:21:36 +00:00
go routine and removing cart 1f6d5ecb72
goc_daniel commented 2026-04-13 13:41:18 +00:00
Author
Owner
Copy Link

this PR is still waiting for review...

this PR is still waiting for review...
dudzic_wiktor approved these changes 2026-04-14 06:28:53 +00:00
goc_daniel added 2 commits 2026-04-14 11:19:53 +00:00
Merge branch 'main' of ssh://git.ma-al.com:8822/goc_daniel/b2b into orders 604247b7c8
fixes after merging with main c610ce38cc
goc_daniel merged commit 5e1a8e898c into main 2026-04-14 11:20:06 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
dudzic_wiktor
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
dudzic_wiktor goc_marek yakovenko_arina yakovenko_valeriia goc_daniel
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: goc_daniel/b2b#58
Delete Branch "orders"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?