Skip to content

Simon's code review #56

@simonryrie

Description

@simonryrie

Love this! It’s a really fun idea for a project. The core functionality works perfectly, which makes me sad because I only have 5 followers :(

Technical:

Is there a link to the deployed site? For this project especially it would be nice to be able to interact with other users!

Is there a reason why dev branch is the default?

Maybe this is a deployment-related issue but when I logged out on dev server then tried to log back in it failed because it tried to take me to this authentication link on the deployed site: https://techyescracy.fly.dev/authenticate?code=d63118978cbaf43b1e2a

Looks like Prettier has missed some indentation clean up on form.js and home.js which makes it a bit hard to follow!

UX:

On the Add Poll section, it might be helpful to add a signpost for the user that it is a yes/no question.

Maybe a bit more styling on Add Poll would be nice

I didn’t understand the relationship between the main pools and the bill of rights sorry!

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions