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

Allow API calls for files not within master branch #4

Closed
landonreed opened this issue Sep 17, 2014 · 7 comments · Fixed by #8
Closed

Allow API calls for files not within master branch #4

landonreed opened this issue Sep 17, 2014 · 7 comments · Fixed by #8

Comments

@landonreed
Copy link

It's common for repos with web apps running on the gh-pages branch to store GeoJSON here rather than in the master branch. Are there plans to add support for API calls to .geojson files in non-master branches?

@landonreed landonreed changed the title Allow API calls for files not within master branch Allow API calls for files not within master branch Sep 17, 2014
@ungoldman
Copy link
Contributor

Hi @landonreed, it's probably too little too late, but I'm working on a full rewrite of this package which should allow for adding your feature request more easily.

@chelm is it possible to transfer this repo over to koopjs and give me npm contributor access (https://www.npmjs.com/package/geohub) since it's solely a koop dependency and is in need of maintenance? I'll do the dirty work. 👷

@chelm
Copy link
Contributor

chelm commented Sep 30, 2015

@ngoldman ahh but of course! Will move it and make you a contributor today, thanks!

@chelm
Copy link
Contributor

chelm commented Sep 30, 2015

@ngoldman 🏁 listo

@ungoldman
Copy link
Contributor

@chelm woooo thanks.

@landonreed not sure if you're still interested but I'll try to get something in to support your use case in the next week or two.

@ungoldman
Copy link
Contributor

Turns out that was pretty easy once I cleaned things up! Should be in the 1.0.0 release later today.

@ungoldman
Copy link
Contributor

@chelm
Copy link
Contributor

chelm commented Oct 1, 2015

@ngoldman 🎯 you rock!

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 a pull request may close this issue.

3 participants