-
Notifications
You must be signed in to change notification settings - Fork 6
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
[#2106, #2203] Implemented 'contactform_confirmation' email #1106
Conversation
c283508
to
3244e41
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.
Rename admin setting
src/open_inwoner/openklant/migrations/0011_openklantconfig_api_sends_email_confirmation.py
Outdated
Show resolved
Hide resolved
…tConfig.api_sends_email_confirmation
3244e41
to
0143e9d
Compare
0143e9d
to
07e9dbd
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #1106 +/- ##
===========================================
+ Coverage 95.05% 95.07% +0.01%
===========================================
Files 950 953 +3
Lines 33790 33920 +130
===========================================
+ Hits 32119 32249 +130
Misses 1671 1671 ☔ View full report in Codecov by Sentry. |
The registration code is pretty spicy. I tried refactoring but it does need many conditionals so except some small changes I kept it this way.