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

GRT Update (Coordinates, Contacts & More Metadata) #2768

Closed
wants to merge 7 commits into from

Conversation

hexylena
Copy link
Member

@hexylena hexylena commented Aug 10, 2016

The backend portion of this is still in development.

  • This follows cloudlaunch's example and uses some external web services to obtain a guess at latitude and longitude. This is completely opt-in and disabled by default.
  • Additionally updates for a requested DNS record change by @martenson.
  • Additionally supports updating the registered name, description, and adding tags to a galaxy instance. Previously this was done via the web interface but moving this into configuration seems like a good choice :)

This is really to support galaxyproject/galaxy-maps#6 as I think GRT is the right place to do this.

TODO:

return {
'lat': json_geoloc['lat'],
'lon': json_geoloc['lon'],
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For this? CloudLaunch? :)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yep, I remembered your PR to coudlaunch, saved me some time since you already had examples of using the service :)

@hexylena hexylena changed the title GRT Update supporting sharing geographic coordinates and instance tags GRT Update (Coordinates, Contacts & More Metadata) Aug 16, 2016
@hexylena hexylena modified the milestones: 17.01, 16.10 Nov 16, 2016
@martenson martenson modified the milestones: 17.05, 17.01 Jan 12, 2017
@hexylena hexylena added this to the 17.09 milestone Apr 26, 2017
@hexylena
Copy link
Member Author

Closing this in favour of a new PR that will be out within a week or two.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants