hackforla / hackforla/peopledepot

epic: Fix Event model `must_attend`, `should_attend`, `could_attend` fields

Open
#567 0 comments 0 reactions 0 assignees View on GitHub
complexity: small epic feature: update table p-feature: events PII: TRUE ready for dev lead role: back end s: PD team size: 1pt
Dominant language
Python
Stars
14
Forks
37
Avg merge
9d 15h
Merged PRs (30d)
5

Description

### Overview

We need to fix issues relating to the `must_attend`, `should_attend`, `could_attend` fields of the `Event` model so that it's decently usable. One is they should accept null inputs, making them optional if needed. Another is that they should probably validate the JSON input.

### Action Items

- [ ] issue to allow null for `must_attend`, `should_attend`, `could_attend` fields
- [ ] issue to validate JSON inputs to these fields for `practice_area` and `permission_type`
- [ ] issue to create admin widget for selecting the fields from possible values

### Resources/Instructions
- Spreadsheet: [PD: Table and field explanations, Current Field Permission tab](https://docs.google.com/spreadsheets/d/1x_zZ8JLS2hO-zG0jUocOJmX16jh-DF5dccrd_OEGNZ0/edit?gid=749942651)
- This issue came out of another issue: https://github.com/hackforla/peopledepot/issues/554#issuecomment-3363540665

> - I found a problem where I can't create events with empty lists in any of `must_attend`, `should_attend`, `could_attend` fields. They need to be made into null=True, blank=True in the model to accept `[]` in the JSON input. I think we do want them to be nullable.
>
> - Another thing is these fields are difficult to fill in with the JSON data they expect. It should be possible to make the admin UI widget load the `practice_area` and `permission_type` values for the user to choose from.

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.