hackforla / hackforla/peopledepot

Epic: Permissions implementation

Open
#148 5 comments 0 reactions 0 assignees View on GitHub
complexity: missing dependency epic ethan p-feature: permissions PII: FALSE research role: back end s: org s: PD team s: VRMS size: 5pt
Dominant language
Python
Stars
14
Forks
37
Avg merge
9d 15h
Merged PRs (30d)
5

Description

### Dependencies

- #150 is ongoing
- https://github.com/hackforla/peopledepot/issues/432
- ~#159~ moved to spreadsheet permissions
- ~#149~ ~(@ethanstrominger is implementing a system separate from the builtin django permissions, so #149 should probably be closed as irrelevant).~
- #346

### Overview

We are implementing the permissions system that we need, including field-level permissions

This is a meta issue to keep track of the action issues.

### Action Items

- [ ] #150
- [ ] implement permissions that will support our needs
- [ ] #346
- [x] take #22 out of the ice box and make any adjustments to it
- [ ] release #29 from the ice boxDescription of what's hidden below
- [ ] close this issue as complete

### Discussion

- Django comes with Group and Permission models in django.contrib.auth out of the box. If they don't match up well, then we need to evaluate existing packages and decide on one that will support our requirements best
- we need to define our requirements (what we need) for permissions
- ex. a project lead needs to be able to update the project they are leading (row in the project table for their project), but not be able to update the other projects (rows belonging to other projects).
- ex. a contributor needs to be able to edit their own user profile, but not the user.status field, since that data belongs to the organization, and not the other user profiles.
- more requirements (aka acceptance criteria)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.