Boosqua / Disorder

0 stars 0 forks source link

proposal checklist #1

Open ghost opened 3 years ago

ghost commented 3 years ago

Wiki Page Home

Comments


MVP List

Comments


Database Schema

Comments


Sample State

Comments


Backend Routes

Comments


Frontend Routes

Comments

ghost commented 3 years ago

Hey Omar, here are my comments for schema and mvp list:

  1. Looks like you're missing a bullet under the messages table for an index on server_id
  2. looks like we may be missing a channels table because a channel lives within a server
ghost commented 3 years ago

Hey Omar, here are my comments for sample state:

  1. let's add some more examples under the entities keys
  2. let's make sure that we include messages and channels in the sample state as well
ghost commented 3 years ago

Hey Omar here are my comments for the home page and routes that you made:

  1. on the home page you can get rid of the header that says discord clone design docs
  2. for your backend routes you are not going to need to create restful routes for your messages since those will come up in a payload with the channels. If you have any questions about that just let me know
  3. You gotta do your frontend routes

lol on your sample state