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

Publish sunshine list as api endpoint #71

Closed
KartikTalwar opened this issue Nov 1, 2013 · 3 comments
Closed

Publish sunshine list as api endpoint #71

KartikTalwar opened this issue Nov 1, 2013 · 3 comments
Assignees
@KartikTalwar
Labels
data request enhancement

Comments

@KartikTalwar
Copy link
Contributor

Make the following dataset an API endpoint

https://github.com/uWaterloo/Datasets/tree/master/Salaries
@ghost ghost assigned KartikTalwar Nov 1, 2013
@KartikTalwar
Copy link
Contributor Author

@krisolafson what do you think we should name this. /resources/salaries.json or sunshinelist.json
Since the list will contain data from the previous year, I'm not sure what the best approach to giving the year parameter in url should be. Default to the latest dataset and add ?year=xxxx?
cc/ @axyjo

@axyjo
Copy link

axyjo commented Nov 11, 2013

salaries seems to imply that the list is complete, whereas sunshinelist is presumed to be incomplete. I like the strategy mentioned for separating the different years.

@KartikTalwar
Copy link
Contributor Author

Yeah makes sense. That data is only on people who make > $100k so I guess I'm leaning towards the sunshinelist

@KartikTalwar KartikTalwar mentioned this issue Nov 11, 2013
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@KartikTalwar KartikTalwar

Labels
data request enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@axyjo @KartikTalwar