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

total project restructure #81

Open
wants to merge 47 commits into
base: main
Choose a base branch
from
Open

Conversation

gkhaavik
Copy link
Member

This pull request includes significant refactoring and cleanup across the entire project structure

@gkhaavik gkhaavik linked an issue Dec 30, 2024 that may be closed by this pull request
fixes server can now start up
implementing OrderStateFlow usage inside OrderValidationService
adding OneToMany relation between user and order
entire products module rewrite, not finished
adding capture command
moving payment created event into the domain service
adding payment repository
Adding webhook config persistence
Cleaning up mobilepay integration propeties
Renaming PaymentResponse DTO to InitializedPayment
Renames the BACKEND_HOST environment variable to MOBILEPAY_WEBHOOK_CALLBACK
Fixing docker compose error
Temporarely disabling shippingCost as it doesnt work with mobilepay integration
…ion ID handling

Improvements to webhook error handling
Refactoring event handling and adding PaymentCancelledEventHandler
Fixing validateOrderForPayment method
Updating order status whenever payment gets reserved or captured
Fixing payment capture status updates
…rework

80 6 order and payment flows rework
…t-handling

# Conflicts:
#	src/main/java/com/zenfulcode/commercify/order/application/service/OrderApplicationService.java
#	src/main/java/com/zenfulcode/commercify/payment/application/events/PaymentEventHandler.java
#	src/main/java/com/zenfulcode/commercify/user/domain/model/User.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

Complete project restructure
1 participant