Merge 25 1 into dev#21067
Merged
mvdbeek merged 337 commits intogalaxyproject:devfrom Oct 17, 2025
Merged
Conversation
Applying suggestions from John Davis
…ials and removing service_reference, optional, and multiple from user credentials definition and add optional tag to the credential definition and update related functionality
… the given definition is not found to prevent unnecessary data on listing credentials
…D, and version for better debugging context
… for undefined services and enhance test coverage for invalid credential scenarios
… to avoid using StructuredApp
…r parameters instead of app and job, init ToolApp with install_model and vault
…utes for improved credential management
For improved user guidance and structure
To use computed serviceName
Improves spacing by adding a top margin to the ToolCredentials component for better visual separation from other elements.
Enhances modal with scrollable content and larger size Improves footer layout and button labels for clarity Adjusts styles for better user experience
Adds edit mode for credential sets Integrates confirmation dialog for deletions Improves form validation and error handling Updates UI components for better user experience
Replaces CSS class-based highlighting with a dedicated highlighted prop for better component encapsulation and maintainability. Removes hardcoded styling logic from the parent component and moves the highlighting responsibility to the card component itself.
Adds event.stopPropagation() calls to keyboard event handlers to prevent events from bubbling up to parent elements when the card handles Enter, Space, or other key interactions.
…e-space [25.1] Fix GCard Layout and White Spaces
…ata_function [25.1] Restore .get_metadata function
[25.0] add LAS and LAZ file format
…flow_editor_fixes [25.0] Fix PUT /api/workflows for user defined tools
…t-keyboard-navigation [25.1] Add Keyboard Navigation to History Lists
…session_issue [25.0] Fix `InvalidRequestError` when saving workflow step with dynamic tool
[25.0] New datatype addition: beast.trees datatype
[25.0] add display_in_upload to LAZ and LAS
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Tool credential PR is huge and with (minor) conflicts, letting this run for the tests
How to test the changes?
(Select all options that apply)
License