Update Event table must-could-should

[fyliu]

Overview

We need to update the event table model in django to utilize the practice area table instead of the outdated role table.

• 173 wasn't as complete as we thought it was

Details

The old model issues for this table are

• 14

• 173

Action Items

• [x] Update existing Django model
• [ ] Write a test for the new relationships this model will have with other models (e.g., creating a user and assigning them a set of permissions on a project) if any.
• [ ] Update API end point
• [ ] Update API unit tests
• [ ] Document the endpoint in Swagger

Changes Needed

Columns to Remove

• [x] must_roles - int[] (role_id)
• [x] should_roles - int[] (role_id)
• [x] could_roles - int[] (role_id)

Columns to Add

• [x] must_attend - JSON[]
• [x] should_attend - JSON[]
• [x] could_attend - JSON[]

Additional Changes

• [x] Add example JSON to the documentation (like the one in the ERD)

JSON Example:

[
  {
    practice_area: engineering,
    permission_level: practiceLeadProject
  },
  {
    practice_area: pm,
    permission_level: practiceLeadProject
  }
]

Resources

• See People Depot Resources wiki page for links
  • ERD
  • Table and Field Definitions
  • API Endpoint

Practice Areas:

• Development
• Project Management
• Design
• Professional Development

Permission Types:

• adminGlobal
• adminBrigade
• adminProject
• practiceLeadProject
• practiceLeadJrProject
• memberProject
• memberGeneral

Test suggestions

• create an "All" event for the PeopleDepot project with these attendance suggestions
  • must: adminProject, practiceLeadProject, practiceLeadJrProject
  • should: memberProject
  • could: memberGeneral
• need an example that utilizes Practice Area values

[fyliu]

Questions:

Check the Test Suggestions section to see if that one makes sense.

We need an example usage of these JSON fields that utilizes Practice Area values.

Do all the Permission Types have to have a Practice Area value or only certain ones like practiceLeadProject, practiceLeadJrProject, memberProject need one?