-
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
[#1482] Added 'infix' (tussenvoegsel) to User model #633
Conversation
99895a3
to
025c12c
Compare
Codecov Report
@@ Coverage Diff @@
## develop #633 +/- ##
===========================================
- Coverage 96.51% 96.50% -0.02%
===========================================
Files 600 605 +5
Lines 20684 20792 +108
===========================================
+ Hits 19964 20066 +102
- Misses 720 726 +6
... and 8 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. 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.
Looks good! Let's add the infix field to the profile/edit.html
as well, because we render the fields manually.
@@ -31,15 +30,16 @@ def test_signal_updates_users_data_when_logged_in_via_digid_v_2(self, m): | |||
user.bsn = "999993847" | |||
user.save() | |||
|
|||
updated_user = User.objects.filter(email=user.email) | |||
user.refresh_from_db() |
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.
Thanks for the updated part!
Thanks, weird how I missed that. I appended a commit for it. |
3b433c3
to
5c864b2
Compare
Sorry, but I had to force-push this again because I had the wrong ticket number in the commit log 😞 (I'll leave the branch name because it is not that important after it is merged) |
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.
Van alle Engelse benamingen voor dit veld vind ik 'infix' nog de beste, en ik heb een hoop verschillende gezien
No description provided.