Skip to content

Fix co-applicant permission for Partners, Reviewers, and Community Reviewers#4583

Merged
frjo merged 1 commit intomainfrom
fix/gh-4578-coapplicant-partner
Jun 17, 2025
Merged

Fix co-applicant permission for Partners, Reviewers, and Community Reviewers#4583
frjo merged 1 commit intomainfrom
fix/gh-4578-coapplicant-partner

Conversation

@sandeepsajan0
Copy link
Copy Markdown
Member

@sandeepsajan0 sandeepsajan0 commented Jun 11, 2025

Fixes #4578

These views are using custom permissions along with submission_view permission, and those custom permissions are causing issues. Now, we have checked if a user with the role of Partner, Reviewer, and Community Reviewer isan applicant or a co-applicant will be redirected to Applicant view.

Test Steps

  • Try to invite any role as a co-applicant
  • Accept the invite by become that user and check if submission is accessible or not.

@sandeepsajan0 sandeepsajan0 marked this pull request as ready for review June 11, 2025 13:47
@frjo frjo added Type: Bug Bugs! Things that are broken :-/ Type: Patch Mini change, used in release drafter Status: Needs testing Tickets that need testing/qa Status: Needs dev testing 🧑‍💻 Tasks that should be tested by the dev team labels Jun 12, 2025
@wes-otf wes-otf added Status: Tested - approved for live ✅ and removed Status: Needs testing Tickets that need testing/qa Status: Needs dev testing 🧑‍💻 Tasks that should be tested by the dev team labels Jun 16, 2025
@frjo frjo merged commit 6d34b02 into main Jun 17, 2025
7 checks passed
theskumar added a commit that referenced this pull request Jun 20, 2025
* origin/main:
  Fix report file path issue and duplicate file issue, Fix file path issue for Project and SOW Form (#4580)
  Use xhtml2pdf inplace of reportlab for submission pdf export (#4577)
  Fix co-applicant permission for Partners, Reviewers, and Community Reviewers (#4583)
  Migrate django-fsm to django-viewflow (#4561)
  Delete application comments with the application (#4585)
@theskumar theskumar deleted the fix/gh-4578-coapplicant-partner branch July 20, 2025 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status: Tested - approved for live ✅ Type: Bug Bugs! Things that are broken :-/ Type: Patch Mini change, used in release drafter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Permission denied when a user that has the role of Partner is invited to be a coapplicant

3 participants