Skip to content
This repository has been archived by the owner on Nov 16, 2024. It is now read-only.

CRSE-db-schema #21

Merged
merged 3 commits into from
May 3, 2022
Merged

CRSE-db-schema #21

merged 3 commits into from
May 3, 2022

Conversation

eccentricOrange
Copy link
Owner

@eccentricOrange eccentricOrange commented May 3, 2022 •
edited
Loading

Created schema.

  • It seems to be 2NF, but I've decided against 3NF because I'd rather deal with one primary key per table than a composite one.
  • Added support for logging (including costs)

@eccentricOrange eccentricOrange merged commit 0f98c9c into 20-incorporate-crse-and-reddit-feedback May 3, 2022
@eccentricOrange eccentricOrange deleted the crse-db-schema branch May 3, 2022 12:04
@eccentricOrange eccentricOrange linked an issue May 3, 2022 that may be closed by this pull request
Incorporate CRSE and Reddit feedback #20
Closed
@eccentricOrange eccentricOrange added enhancement New feature or request CRSE Feedback from the Code Review Stack Exchange labels Jun 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
CRSE Feedback from the Code Review Stack Exchange enhancement New feature or request
Projects
None yet
Milestone
First round of feedback from CRSE
Development

Successfully merging this pull request may close these issues.

Incorporate CRSE and Reddit feedback
1 participant
@eccentricOrange