-
Notifications
You must be signed in to change notification settings - Fork 8
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
Release CI #75
Release CI #75
Conversation
The users query is now querying over the ClaimAdmin and Officer types. This is not really a clean solution but it is the best that we could make with the existing table structures without breaking everything else. Be careful not to optimize the graphql query or it would prevent the User dynamic mapping to the actual objects because some of them would be marked as "deferred". The Mutations are not yet part of this commit but will follow shortly.
This requires an update to the Location module.
The "user" in UserMutation was clashing with the operating user.
It is now possible to call the user creation/update method with just the username instead of the exact core_user ID
health_facility_id was of the wrong type and not rendered in the response for InteractiveUser because the field is not a foreign key. Also added roles with a direct rendering rather than through a userRoles intermediate object.
The mutation link to actual objects was performed on the actual object itself. In the case of User, it caused some conflicts with its `__call__` method because _u was not properly initialised from Django itself. So we are now adding _id to the object and passing its id. We still need the object itself to identify the foreign key. UserMutation was also not exposed to GraphQL and the user was not properly linked for the mutation, it was linked to connected user.
This was preventing the delete of users created from the mutations.
OFS-290: fixed bug with performing uuids bulk actions
OTC-289: Added GQL Query for languages
OFS-292: fixing bulk actions with amount/date valid filters
OE0-4: fixed getModulePermissions in core module
Json configuration loaded as OrderedDict
OE0-64: Fixed HistoryModel uuid property
Save more fields of User, Officer & IUser when using Django
OE0-29: added migration for put missing role for ClaimAdministrator
#68 health_facility_id was renamed in code but not tests The districts were added but didn't account for users without district limits.
This pull request introduces 5 alerts and fixes 6 when merging 98263c3 into d02ac16 - view on LGTM.com new alerts:
fixed alerts:
|
Details: GHSA-xpfp-f569-q3p2 The proper way to address this would be to upgrade Django to 3.1.13 but it requires to update all the JSONField (and convert the NullBooleanField to BooleanField). While the JSONField seems to work when we remove the JSONFallback component, the boolean field does not work anymore with `bit` data types. They are compared with the boolean instead of 0 or 1. I tried to check for updates on the mssql driver but it seems not to have moved in a long while. I have therefore implemented the same kind of fix as the 3.1.13 did, right into the OrderedDjangoConnectionField...
OPL-13: Added decorators and functions for service signals
This pull request introduces 5 alerts and fixes 6 when merging 1083a53 into d02ac16 - view on LGTM.com new alerts:
fixed alerts:
|
This pull request introduces 5 alerts and fixes 6 when merging a58a7d8 into d02ac16 - view on LGTM.com new alerts:
fixed alerts:
|
OPL-6: Fixed Base Delete Mutation
Security fix for Django order_by flaw
This pull request introduces 5 alerts and fixes 6 when merging d9f1d10 into d02ac16 - view on LGTM.com new alerts:
fixed alerts:
|
Fix field used to get db User
This pull request introduces 5 alerts and fixes 6 when merging aafe671 into d02ac16 - view on LGTM.com new alerts:
fixed alerts:
|
No description provided.