informed-governance-project / NISINP

Incident Notification Platform by @NC3-LU
https://nisinp.readthedocs.io
GNU Affero General Public License v3.0
6 stars 3 forks source link

PlatformAdmin should see all users and companies #133

Open tochinet opened 7 months ago

tochinet commented 7 months ago

At the moment, the platformAdmin role is limited to CRUD RegulatorAdmin (and other PlatformAdmin). But we can expect that all the RegulatorAdmin(s) and RegulatorUser(s) will have (at least for the first months after onboarding) limited experience and could need some help to correct errors (like users without affiliation, etc.)

The simple way to solve that is to allow the PlatformAdmin to CRUD all "regulations", "regulators", "sectors" (like now), but also "companies" and "users", including the affiliation of users to companies or regulators + "admin" role, but excluding mix of both.

There should be no access to incidents for the PlatformAdmin.

At this moment, the users created by the PlatformAdmin have no association atall, hence theses users can login, but are "outside the protocol" and can't do anything useful. Correcting this and other errors shouldn't imply a manual modification in the database !

tochinet commented 1 month ago

This has changed since the release that was tested when this issue was created.