-
Notifications
You must be signed in to change notification settings - Fork 2
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
✨(dashboard) add signals on delivery point creation #307
Conversation
cd543e6
to
bc33710
Compare
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.
Good to know that signals management evolved since I used them!
f028777
to
0df1e8b
Compare
0df1e8b
to
e18d9df
Compare
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.
I have minor remarks to refine tests! Keep it up! 💪
92936b5
to
9fd7a85
Compare
9fd7a85
to
fd5c01c
Compare
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.
Implemented a signal to automatically create a Consent object when a new DeliveryPoint is saved. Added corresponding tests to ensure functionality and updated related Pipfile dependencies. Centralized consent end date logic in a new app settings file (CONSENT_END_DATE) and updated all related signals, factories, and tests to use it. Updated documentation and code comments to reflect these changes.
fd5c01c
to
2149df1
Compare
Purpose
When a new
delivery point
is created, the associatedconsent
must be automatically created as awaiting consent.Proposal