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

Feature/summarize by date #23

Merged
merged 8 commits into from
Jan 30, 2015
Merged

Conversation

israelst
Copy link
Contributor

Related to #19.
Better merged after (or replacing) #22.

@fccoelho
Copy link
Member

@israelst, I don't see any changes to the interface, what's missing?

@israelst
Copy link
Contributor Author

I don't know why github is showing these first four commits that belongs to #22... :/
This PR just create the routes and group the data, the following one will add our new chart.

@israelst
Copy link
Contributor Author

@fccoelho, you can see what is new by getting /incidents/summary/date

fccoelho added a commit that referenced this pull request Jan 30, 2015
@fccoelho fccoelho merged commit 5eed0bf into NAMD:master Jan 30, 2015
@israelst israelst deleted the feature/summarize-by-date branch January 30, 2015 14:16
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

Successfully merging this pull request may close these issues.

2 participants