jeffdeliso / discors Goto Github PK

nvm

Hello! I was just wondering how I would get this up and running. What I mean by that is what commands would I use to properly set this up.

Thanks in advance!

Leaves site at a 503 error

###Frontend State

• Our slice of state for currentUser should just be an id pointing to the users slice of state.

###Backend Routes

• GET request for server_memberships is unnecessary, we should just pass that information down on login. Same goes for servers and friendships, assuming that we can't view servers when not logged in. I'd say that you probably do want to make a separate request for messages though.

Great job so far! Some of the things that I'm going to list aren't issues per say, but suggestions that I'll leave up to you if you want to change/implement.

• We don't want any dead links in our apps. The Download for macOS button should either be changed to something else or removed.

• If the settings cog in the bottom left isn't going to have a separate menu pop-up, you may want it to be more obviously a logout button.

• There should be an ability to leave servers that you are not an admin of.

• In your MVP's you list that only an admin of a channel should be able to create text channels. However, i t may be good to keep the functionality it as is for demoing purposes. Feel free to amend your MVP's if you decide against it.

• There is currently no ability to delete text channels (as far as I can tell). If anyone can create a text channel, at the very least the admin should be able to delete them.

• Currently no ability to accept friend requests.

would love it , if you add some kind of instructions !

Returns https://discors.herokuapp.com/api/session 500 (Internal Server Error) while logging in

Note that all of these issues are based on the version live on heroku as of 1/23 4:30pm

• It looks like fonts are not being imported correctly.

• Image assets are not loading in. Lots of 404 errors.

• The "Need an account? Register" portion is missing space between "?" and "Register"

• Slight vertical spacing issues within the form, but that may also be affected by the change in font.