mirror of
https://github.com/cse110-fa22-group29/cse110-fa22-group29.git
synced 2024-12-26 00:49:10 +00:00
Merge pull request #100 from cse110-fa22-group29/backend-adrs
important backend adrs
This commit is contained in:
commit
842c703d1a
16
specs/adrs/112922-review-storage.md
Normal file
16
specs/adrs/112922-review-storage.md
Normal file
@ -0,0 +1,16 @@
|
||||
# Backend Storage Structure
|
||||
- Status: Accept
|
||||
- Deciders: Rhea Bhutada, Kara Hoagland, Gavyn Ezell, George Dubinin, Henry Feng
|
||||
- Date: 11/29/2022
|
||||
|
||||
## Decision Drivers
|
||||
- Needed more efficient way of storing reviews that are created, for more efficient testing, updating, accessing, and deleting.
|
||||
|
||||
## Considered Options
|
||||
- localStorage
|
||||
|
||||
## Decision Outcome
|
||||
Using local storage to maintain the "local first" requirement.
|
||||
Moved away from array of objects for storing reviews, reviews are stored individually as keys in localStorage, under the "review{id}" format. Each key
|
||||
corresponds to object containing review data. We also have an array stored in local storage, named "activeIDs" which keeps track of id numbers that are attached
|
||||
to created reviews.
|
15
specs/adrs/112922-reviewpage-session.md
Normal file
15
specs/adrs/112922-reviewpage-session.md
Normal file
@ -0,0 +1,15 @@
|
||||
# Opening Specific Reviews
|
||||
- Status: Accept
|
||||
- Deciders: Rhea Bhutada, Kara Hoagland, Gavyn Ezell, George Dubinin, Henry Feng
|
||||
- Date: 11/29/2022
|
||||
|
||||
## Decision Drivers
|
||||
- When opening up a review, browser needs to know what review ID to use for loading the review page data
|
||||
|
||||
## Considered Options
|
||||
- sessionStorage
|
||||
|
||||
## Decision Outcome
|
||||
Review cards have event listeners that will add their associated review ID number to session storage so
|
||||
when the review loads, the browser will use the id stored to pull exact data corresponding to the review.
|
||||
|
13
specs/adrs/112922-tag-review-collision-fix.md
Normal file
13
specs/adrs/112922-tag-review-collision-fix.md
Normal file
@ -0,0 +1,13 @@
|
||||
# Organizing Review Under Tags
|
||||
- Status: Accept
|
||||
- Deciders: Rhea Bhutada, Kara Hoagland, Gavyn Ezell, George Dubinin, Henry Feng
|
||||
- Date: 11/29/2022
|
||||
|
||||
## Decision Drivers
|
||||
- Needed to keep track of reviews under certain given tags for filtering feature.
|
||||
|
||||
## Considered Options
|
||||
- localStorage
|
||||
|
||||
## Decision Outcome
|
||||
For every tag create a key under that tag name in localStorage. They will store an array of IDs that correspond to reviews that contain that tag.
|
Loading…
Reference in New Issue
Block a user