Skip to content
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

Tracking pull request to merge release-1.64.0 to master #2328

Merged
merged 17 commits into from
Nov 19, 2024

Conversation

kuanfandevops
Copy link
Collaborator

No description provided.

kuanfandevops and others added 17 commits October 22, 2024 10:57
* initial commit

* cleanup

* more cleanup

* set default permissions to AllowNone

* more cleanup
* chore: removes get_serializer_context

* -refactors mixin and updates the files that use it
-adds extra check to credit transfers details page

* -fixes rebase issues

* chore:  removes commented code

* chore: adds mixin to sales submission list serializer so it passes test

* some fixes and changes

* some refactoring

---------

Co-authored-by: tim738745 <[email protected]>
* chore: tests for user deetails and update

* moves comments

* checks for is government before displaying supplier details page (idir view)

* chore: organization tests
tasks: some changes to organization viewset , adds permissions
fix: adds check for is government on user viewset for download_active
* fix: adds user id to basic mixin so comments can be edited
chore: testing

* removing extra lines

* space
fix: grabs supplementary comments also using supplementary id instead of just reassessment id
chore: adds empty strings to default value in user details form so there are  no console errors if there are blank fields
@kuanfandevops kuanfandevops merged commit c14cd42 into master Nov 19, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants