Skip to content

Routes setup - #7

Open
jsiems wants to merge 11 commits into
2.0from
routes_setup
Open

Routes setup#7
jsiems wants to merge 11 commits into
2.0from
routes_setup

Conversation

@jsiems

@jsiems jsiems commented Aug 5, 2016

Copy link
Copy Markdown

Adds some basic routes.

NOTES:
All data manipulation happens in the controller. This is temporary, only there to test functionality of the routes. All of the data manipulation will be replaced by the model, which is being worked on by Ben.
Also some of the names of the routes aren't exactly great.

I think that this branch sets up the rest of the program pretty well, but it is definitely not perfect. From this branch, it should be easy to add more features or polish existing ones, using the existing routes and code as an example, unless I actually did an absolute terrible job writing this code... which is very possible :)

@jsiems

jsiems commented Aug 5, 2016

Copy link
Copy Markdown
Author

Also, it sends the 'top' customers, locations and asset types when the user requests an asset. These are not actually the top ones, rather just the first 4 that sql decides to pull out from the database.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants