Use tahoe_sites.api.get_organization_for_user - Githubissues

appsembler / course-access-groups

An Open edX plugin to customize courses access by grouping learners and assigning different permissions to groups.

https://course-access-groups.readthedocs.io/

MIT License

4 stars 3 forks source link

Use tahoe_sites.api.get_organization_for_user #107

Closed shadinaif closed 2 years ago

shadinaif commented 2 years ago

Description:

Use tahoe_sites.api.get_organization_for_user
Fix is_organization_staff multi-organizations issue

JIRA:

Dependencies: dependencies on other outstanding PRs, issues, etc.

Merge deadline: List merge deadline (if any)

Installation instructions: List any non-trivial installation instructions.

Testing instructions:

Open page A
Do thing B
Expect C to happen
If D happened instead - check failed.

Reviewers:

[ ] tag reviewer
[ ] tag reviewer

Merge checklist:

[ ] All reviewers approved
[ ] CI build is green
[ ] Version bumped
[ ] Changelog record added
[ ] Documentation updated (not only docstrings)
[ ] Commits are squashed

Post merge:

[ ] Create a tag
[ ] Check new version is pushed to PyPI after tag-triggered build is finished.
[ ] Delete working branch (if not needed anymore)

Author concerns: List any concerns about this PR - inelegant solutions, hacks, quick-and-dirty implementations, concerns about migrations, etc.

shadinaif commented 2 years ago

@OmarIthawi please double-check the fix for https://github.com/appsembler/course-access-groups/blob/c81f2faf53d74f218eba427073b7d74a13f0b125/course_access_groups/permissions.py#L44