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

Enable .KMZ export for Advanced Search results #1495

Closed
leovegoda opened this issue Dec 13, 2023 · 16 comments · Fixed by #1564
Closed

Enable .KMZ export for Advanced Search results #1495

leovegoda opened this issue Dec 13, 2023 · 16 comments · Fixed by #1564
Assignees
Labels
Time:Minor Up to 4 hours

Comments

@leovegoda
Copy link
Contributor

Is your feature request related to a problem? Please describe.
The Advanced Search provides geographic results for Campuses, Exchanges, Facilities and Organizations. These can be exported as JSON or CSV. We should add .KMZ as an export option now we share all facility data in .KMZ format.

Who is affected by the problem?
Anyone who wants to look at data on a map.

What is the impact?
Improved service offering.

Are there security concerns?
No. This is the same data in a new format.

Are there privacy concerns?
No.

Describe the solution you'd like
Add a .KMZ export option.

Do you think this feature will require a formal design?
Possibly.

Describe alternatives you've considered
Do nothing.

Could this feature request need support from the Admin Committee?
It should not.

What is the proposed priority?
Implement in a .KMZ focused release early in 2024

@leovegoda leovegoda self-assigned this Dec 13, 2023
@leovegoda leovegoda added this to the 1 Decide milestone Dec 13, 2023
@mcmanuss8
Copy link
Contributor

+1

@martinhannigan
Copy link

martinhannigan commented Dec 13, 2023 via email

@leovegoda
Copy link
Contributor Author

How much resource is this going to use?

Probably very similar to what we use now.

I would suggest we at least put this function behind a login account.

All the geographic use of Advanced Search already requires users to be logged in. This is just a new way to package the same results they are already getting.

@leovegoda leovegoda reopened this Dec 13, 2023
@martinhannigan
Copy link

martinhannigan commented Dec 13, 2023 via email

@leovegoda
Copy link
Contributor Author

The @peeringdb/oc can comment on the CPU requirements to generate a .KMZ file versus JSON.

All our data, except for contact data marked as private, is already available in other formats. You can sync PeeringDB to your laptop with https://github.com/peeringdb/peeringdb-py. We could add a login requirement but that would a policy change.

@martinhannigan
Copy link

martinhannigan commented Dec 13, 2023 via email

@grizz
Copy link
Member

grizz commented Dec 14, 2023

+1

@leovegoda
Copy link
Contributor Author

So moving the kmz from the main page/www without auth to a page that requires auth is a policy change?

Requiring a login for data that does not require a login now sounds like a policy change to me. What is the advantage we get from making them login?

@martinhannigan
Copy link

martinhannigan commented Dec 14, 2023 via email

@leovegoda
Copy link
Contributor Author

I think the decision to put it in the footer was taken by me and 20C. We chose that because it just repackaged existing content that was already available anonymously.

@martinhannigan
Copy link

martinhannigan commented Dec 14, 2023 via email

@martinhannigan
Copy link

martinhannigan commented Dec 15, 2023 via email

@grizz
Copy link
Member

grizz commented Dec 18, 2023

outputting to KML is going to be very similar to CSV. KMZ is compressed, so it would be a bit more.

@martinhannigan
Copy link

-1

@martinhannigan
Copy link

+1

1 similar comment
@jackcarrozzo
Copy link

+1

@vegu vegu added the Time:Minor Up to 4 hours label Feb 6, 2024
vegu pushed a commit that referenced this issue Mar 12, 2024
v2 search - support for ISO 3166 alpha-2 country codes - fixed #1447
Enable .KMZ export for Advanced Search results - fixed #1495
BFD support field in Global and IX specific views - fixed #1331
Remove unneeded fields from the KMZ - fixed #1489
Return auth error when multiple auth methods are used - fixed #1133
Social link controls showing up when not logged in - fixed #1478
Tab URLs don't work anymore - fixed #1152
Update social media icons in footer - fixed #1425
@vegu vegu mentioned this issue Mar 12, 2024
@grizz grizz closed this as completed in 91149f0 Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Time:Minor Up to 4 hours
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants