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

Support new Galaxy configuration setting interactivetoolsproxy_map #120

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

kysrpex
Copy link

@kysrpex kysrpex commented Jul 18, 2024

At the moment, the Gravity setting sessions is "overridden" with interactivetools_map if configured in the Galaxy settings.

A new Galaxy setting interactivetoolsproxy_map was added in galaxyproject/galaxy#18481 that overrides interactivetools_map` if defined. Gravity should do the same. This PR also documents the change.

…hemy`

At the moment, the Gravity setting `sessions` is "overridden" with `interactivetools_map` if configured in the Galaxy settings.

A new Galaxy setting `interactivetools_map_sqlalchemy` was added in Galaxy PR #18481 that overrides interactivetools_map` if defined. Gravity should do the same. This commit also documents the change.
@kysrpex kysrpex changed the title Support new Galaxy configuration setting interactivetools_map_sqlalchemy Support new Galaxy configuration setting interactivetoolsproxy_map Sep 5, 2024
@kysrpex kysrpex marked this pull request as ready for review October 29, 2024 07:31
@kysrpex
Copy link
Author

kysrpex commented Oct 29, 2024

@natefoo All the required functionality has now been merged to Galaxy (galaxyproject/galaxy#18481), the Interactive Tools Proxy (galaxyproject/gx-it-proxy#21, galaxyproject/gx-it-proxy#22) and an updated tutorial to the GTN (galaxyproject/training-material#5179) is also ready to be merged, it would now make sense to merge 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

Successfully merging this pull request may close these issues.

1 participant