Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cleanup code #1

Open
serverwentdown opened this issue Oct 9, 2016 · 1 comment
Open

Cleanup code #1

serverwentdown opened this issue Oct 9, 2016 · 1 comment

Comments

@serverwentdown
Copy link
Owner

Currently, code is a monolithic blob of routes without abstractions.

Migrate everything to helper promises and map()s.

@serverwentdown
Copy link
Owner Author

  • Have proper error handling
    • Shared default error handler, that will always do req.flash()
  • Switch to async/await syntax, and Promises APIs
  • Split across multiple files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant