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: event_history #49

Closed
24 of 33 tasks
Tracked by #181
ericvennemeyer opened this issue Sep 8, 2022 · 5 comments
Closed
24 of 33 tasks
Tracked by #181

Create Table: event_history #49

ericvennemeyer opened this issue Sep 8, 2022 · 5 comments

Comments

@ericvennemeyer
Copy link
Member

ericvennemeyer commented Sep 8, 2022

Overview

We need to create the event_history 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

  • decide if we're using django-simple-history for this table
  • 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

A history of events

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 - Record Id
    • (FK) event_id - int - Record Id
    • (FK) project_id - int - Record Id
    • description - varchar
    • video_conference_url - url varchar
    • (FK) location_id - int - Record Id
    • alt_location_name - varchar
    • alt_address1 - varchar
    • alt_address2 - varchar
    • alt_location_city - varchar
    • alt_location_state - varchar(2)
    • alt_location_zip - varchar
    • start_date_time - timestamp
    • duration_in_min - int
    • (FK) recurring_event_id - int - Record Id
    • created - timestamp
    • check_in_ready - boolean
  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)

    • location (many-to-one)
    • project (many-to-one)
    • recurring_event (many-to-one)
    • event (implied one-to-one)
  2. In ERD only (having items here indicates a mismatch, which requires a review)

    • None
@fyliu fyliu mentioned this issue Sep 11, 2022
39 tasks
@fyliu fyliu added this to the All tables added milestone Sep 25, 2022
@ExperimentsInHonesty ExperimentsInHonesty added p-feature: events s: VRMS stakeholder: VRMS s: hackforla.org stakeholder: hackforla.org website and removed MVP-1 labels Mar 4, 2023
@ExperimentsInHonesty
Copy link
Member

ExperimentsInHonesty commented Mar 31, 2023

@Neecolaa will create a requirements for this so @fyliu can evaluate the plug in

@fyliu
Copy link
Member

fyliu commented Mar 31, 2023

django-simple-history is the plug-in Django app

@fyliu
Copy link
Member

fyliu commented Aug 3, 2023

@brdeleon this issue is not ready for backend to work on yet. Please find one that has the smiley emoji in the title.

You can tell if it's ready for backend by looking at the "role: ..." tag in the issue. I see there's only one ready issue left. I'll check the board to see if more issues are ready to work on.

@brdeleon brdeleon removed their assignment Aug 3, 2023
@fyliu
Copy link
Member

fyliu commented Aug 11, 2023

This table is being removed in the new event schema design

The ERD and spreadsheet is already changed to remove this table.

There's no need to remove code since this one was not yet coded.

We should move any unfinished tasks into new issues and close this one:

@fyliu
Copy link
Member

fyliu commented Aug 11, 2023

Closing this since there's nothing more to do. This table has be designed out because we no longer want to have a table to hold past individual event instances. Our new design will have recurrences information in the event itself.

@fyliu fyliu closed this as not planned Won't fix, can't repro, duplicate, stale Aug 11, 2023
@shmonks shmonks moved this to Done in P: PD: Project Board Jun 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: ✅Done
Development

No branches or pull requests

4 participants