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

Refactor api structure #10

Open
scuerda opened this issue Jul 6, 2017 · 0 comments
Open

Refactor api structure #10

scuerda opened this issue Jul 6, 2017 · 0 comments

Comments

@scuerda
Copy link
Contributor

scuerda commented Jul 6, 2017

The scraping targets are currently defined within a JSON that needs to be updated when changes are made. If EdSight endpoints were consistently defined and/or structured this would be reasonable, but a number of datasets have specific issues ranging from factor naming typos to alternate ways of presenting the report endpoints to the user.

For example, the state-level year-specific enrollment data is formatted closer to a print report, with multiple CSVs served from one page and subgroups disabled.

Should consider moving internal functions for generating URLs to be dataset specific and adapt a registration approach similar to what is used in the town profile proxy API. This could be handled incrementally by checking dataset by name and routing accordingly.

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