-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Description
Database Schema
- Contains correct datatypes
- Contains appropriate constraints/details
- primary key
- not null
- unique
- indexed
- foreign key
- Contains bullet points after the table that state which foreign keys will reference to which table, or references to the associations which will be made
- foreign key and table name are lowercased, snake_cased and
back_ticked
- foreign key and table name are lowercased, snake_cased and
- Correctly formatted
- schema is written in a table format
- the table's name are lowercased, snake_cased and
back_ticked - the table header column names are bolded
- columns names are lowercased and snaked_cased and
back_ticked
Comments
Anuj looks great, only real comment is you should probably change the name of user_id in songs to artist_id or something a little more semantic just for clarity.
Backend Routes
- Contains the following sections: HTML, API Endpoints(Backend)
- Each route has a description
- API Endpoint routes contains wildcard variables written in
snake_case - Routes does not contain superfluous routes
- Have API routes that will allow the front end to get all info it needs and does not have unneeded routes:
- i.e. probably don't need a
GET likesapi endpoint because that info comes through the post show
- i.e. probably don't need a
Comments
For your GET users route you probably only need to get a specific user not all users so it should be GET /api/users/:id. Otherwise looks great!
Sample State
- State shape is flat!
- State's keys are camelCased
- All keys within the values in the state are accessible in the schema
- Correctly formatted
-[x] Sample state is rendered with triple backticks, and the language(```javascript...```). This will display the state as a code block instead of a giant line of text- Top level slices
-
entities -
session -
errors(here or inui) -
ui(if needed)
-
- Should NOT have nested slices, aka
commentsinside ofposts- Some info from other tables is ok, for instance:
- the author username and imageurl for a post. basically any info that the user can't change
- like count and a boolean on whether the user likes the post instead of a likes slice
- Some info from other tables is ok, for instance:
- Top level slices
Comments
Hey Anuj I saw that users have a profilePic in the sample state but not in the schema.
Frontend Routes
- Frontend routes contains wildcard variables written in
camelCase - Correctly formatted
- Routes are displayed with inline coding text (backticks)
- Routes are displayed with inline coding text (backticks)
Comments
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels