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

two-way integration #5

Open
incanus opened this issue Dec 16, 2013 · 6 comments
Open

two-way integration #5

incanus opened this issue Dec 16, 2013 · 6 comments

Comments

@incanus
Copy link
Contributor

incanus commented Dec 16, 2013

Show latest upload gists in menu for easy recall. Relies on #2 menu.

@incanus
Copy link
Contributor Author

incanus commented Dec 18, 2013

No API for this as of yet -- not going to be possible to get resulting geojson.io URL after processing.

@tmcw
Copy link

tmcw commented Dec 18, 2013

Hm, no API for which part? Do you mean GitHub Gists? If you post to gist instead of using the data-uri approach, you can know the resulting URLs (and also have a much higher max file size)

@incanus
Copy link
Contributor Author

incanus commented Dec 18, 2013

Ah, I wasn't aware of this. So if I make a gist of GeoJSON data, pass that to id=gist:, the resulting geojson.io map will be the same id? What about if doing #3 other file formats? Data would go in as one gist (e.g. KML) and would come out as another (GeoJSON).

@tmcw
Copy link

tmcw commented Dec 18, 2013

I'd rather keep the import API simple and not have it autoconvert Gist contents; the complexity of geojson.io needs to be relatively directed or it will creep.

@incanus
Copy link
Contributor Author

incanus commented Dec 18, 2013

Then I'm not sure I understand:

If you post to gist instead of using the data-uri approach, you can know the resulting URLs

Unless you just mean that two-way will only work for GeoJSON (which may be what you're saying).

That leads to a scenario when only your most recent GeoJSON uploads are in the menu, which is sort of confusing.

@incanus
Copy link
Contributor Author

incanus commented Dec 19, 2013

Unless you just mean that two-way will only work for GeoJSON (which may be what you're saying).

Doing this.

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

2 participants