BCStudentSoftwareDevTeam / BCStudentSoftwareDevTeam/celts

isAdmin property verification

Open
#1,697 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
1
Forks
15
Avg merge
5d 39m
Merged PRs (30d)
8

Description

In app/models/user.py in development, we have this:

Image

We need to investigate and make sure that we are not giving permissions that should not be given, double-check if we actually need isCeltsStudentStaff to be included as admin instead of isProgramManagerFor for instance, because if anything program managers should have more access than simple student staff. And we should make sure that if we need to remove isCeltsStudentStaff it will not create any issues.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.