Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create Table: user_check #80

Open
15 of 20 tasks
fyliu opened this issue Sep 12, 2022 · 3 comments
Open
15 of 20 tasks

Create Table: user_check #80

fyliu opened this issue Sep 12, 2022 · 3 comments
Labels
complexity: missing draft This issue is not fully-written feature: table creation needs FK text see issue #150 for explanation, temporary label delete when done p-feature: onboarding p-feature: user PII: TRUE Table or function has or uses Personally Identifiable Information (PII) ready for db architect role: back end s: CTJ stakeholder: Civic Tech Jobs size: 2pt Can be done in 7-12 hours

Comments

@fyliu
Copy link
Member

fyliu commented Sep 12, 2022

Overview

We need to create the user_check table so that we can update a shared data store across hackforla.org, vrms, civictechjobs, and tables (onboarding) project.

Details

A table and a model are the same thing

Action Items

  • identify and document table description (see spreadsheet under Resources)
    • if not, reach out to PD leads
  • compare and check off the data fields (below) against the ERD. Note any fields in the ERD that's missing from the list. (see Resources)
  • compare the check off associated tables (below) against the ERD. Note any associated tables in the ERD that's missing from the list (see Resources)
  • create a single model in Django (defining schema)
  • write a test for the relationships this model will have with other models (e.g., creating a user and assigning them a set of permissions on a project).
  • write an API end point
  • write API unit tests
  • document the endpoint

Resources/Instructions

Description

Missing

  • this is a generic table that holds checks that are asked of users (Accepted Code of Conduct, Github membership public, Google Calender invite Accepted, ...)

Data Fields

  1. Copied from spreadsheet and checked off according to ERD. (unchecked items indicate a mismatch between ERD and spreadsheet, which requires a review)

    • (PK) id - int
    • (FK) user_id - int
    • (FK) org_id - int
    • (FK) project_id - int
    • (FK) check_type_id - int
    • check_result - Boolean
    • reminder_start - timestamp
    • completion - timestamp
  2. In ERD only (having items here indicates a mismatch, which requires a review)

    • None

Associated Tables

  1. Copied from spreadsheet and checked off according to ERD. (unchecked items indicate a mismatch between ERD and spreadsheet, which requires a review)

    • check_type (many-to-one)
    • organization (many-to-one)
    • user (many-to-one)
  2. In ERD only (having items here indicates a mismatch, which requires a review)

    • project (many-to-one)
@fyliu
Copy link
Member Author

fyliu commented Sep 12, 2022

Problems:

  1. Missing description
  2. Missing table association in spreadsheet
    1. I added the table and join type to the spreadsheet.

@fyliu fyliu added this to the All tables added milestone Sep 25, 2022
@ExperimentsInHonesty ExperimentsInHonesty added the PII: TRUE Table or function has or uses Personally Identifiable Information (PII) label May 4, 2023
@ExperimentsInHonesty
Copy link
Member

@Neecolaa Please review and see if we are ready to proceed with making this table.

@Neecolaa
Copy link
Member

Neecolaa commented Mar 1, 2024

Updated brigade to organization, still need to check project connection before releasing issue.

@shmonks shmonks moved this to Questions/Review in P: PD: Project Board Jun 7, 2024
@ExperimentsInHonesty ExperimentsInHonesty moved this from ❓Questions/Review to 🆕New Issue Review in P: PD: Project Board Sep 6, 2024
@ExperimentsInHonesty ExperimentsInHonesty added the draft This issue is not fully-written label Sep 6, 2024
@ExperimentsInHonesty ExperimentsInHonesty added the needs FK text see issue #150 for explanation, temporary label delete when done label Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
complexity: missing draft This issue is not fully-written feature: table creation needs FK text see issue #150 for explanation, temporary label delete when done p-feature: onboarding p-feature: user PII: TRUE Table or function has or uses Personally Identifiable Information (PII) ready for db architect role: back end s: CTJ stakeholder: Civic Tech Jobs size: 2pt Can be done in 7-12 hours
Projects
Status: 🆕New Issue Review
Development

No branches or pull requests

3 participants