-
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
Update fields data type #2139
Update fields data type #2139
Conversation
61de511
to
04f3789
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 idea to remove character limit.
04f3789
to
1e82e6f
Compare
b5d6520
to
cf5c16f
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.
Let's optimize this on next PR
country_admin_ids = [ | ||
codename.replace('country_admin_', '') | ||
for codename in Permission.objects.filter( | ||
group__user=user, | ||
codename__startswith='country_admin_', | ||
).values_list('codename', flat=True) | ||
] | ||
if user.is_superuser or object.country_id in country_admin_ids: |
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.
country_admin_ids = [ | |
codename.replace('country_admin_', '') | |
for codename in Permission.objects.filter( | |
group__user=user, | |
codename__startswith='country_admin_', | |
).values_list('codename', flat=True) | |
] | |
if user.is_superuser or object.country_id in country_admin_ids: | |
if user.is_superuser: | |
return True | |
country_admin_ids = [ | |
codename.replace('country_admin_', '') | |
for codename in Permission.objects.filter( | |
group__user=user, | |
codename__startswith='country_admin_', | |
).values_list('codename', flat=True) | |
] | |
if object.country_id in country_admin_ids: |
Addresses
Health Data
dropdown. Addshow more
#2140Changes
CharField
replaced withTextField
where ever possibleChecklist
File link
Release
If there is a version update, make sure to tag the repository with the latest version.