29th / personnel-v3

Personnel management system version 3
https://www.29th.org
2 stars 6 forks source link

Use 'manage' permission to determine access to /manage/ section #228

Closed wilson29thid closed 7 months ago

wilson29thid commented 7 months ago

This is an alternative to #226 as a way to solve #222

I've discussed with @swomma and we think this is simpler. Certainly fewer changes as well.

Note that this PR also adds data to the database, via a data migration. It creates 60-some unit permission records, adding the manage permission to every unit that has some other permission already (ignoring event_aar*). It uses the access_level of the lowest permission that the unit has (again, ignoring event_aar*).

Once this is merged, we'll need to run the following command on the production console:

bin/rails data:migrate