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.

Sign up for GitHub

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

Jump to bottom

Clean up node code #121

Closed
6 of 10 tasks
ivarconr opened this issue Apr 27, 2016 · 2 comments
Closed
6 of 10 tasks

Clean up node code #121

ivarconr opened this issue Apr 27, 2016 · 2 comments
Milestone
unleash "next"

Comments

@ivarconr
Copy link
Member

ivarconr commented Apr 27, 2016 •
edited
Loading

  • Clean up the express code
    • All routes in separate folder
    • All db-code in separate folder
    • All config set up in one place, use nconfig or similar
    • All required config injected to routes (database abstraction injected from top)
  • Upgrade to Node 6 Node 5
  • upgrade all libraries to latest version
    • This does not include frontend-dep such as react, redux.
  • Switch to native promises (drop bluebird)
@Andersos Andersos added this to the 2.0 milestone Apr 27, 2016
@ivarconr
Copy link
Member Author

Working branch is: https://github.com/finn-no/unleash/tree/cleanup_api

@ivarconr
Copy link
Member Author

ivarconr commented May 1, 2016

Will work further on this issue under PR: #127

@ivarconr ivarconr closed this as completed May 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
unleash "next"
Development

No branches or pull requests
2 participants
@ivarconr @Andersos