Skip to content

Remove History Panel proxy#21037

Merged
mvdbeek merged 14 commits intogalaxyproject:devfrom
guerler:remove_jquery
Oct 14, 2025
Merged

Remove History Panel proxy#21037
mvdbeek merged 14 commits intogalaxyproject:devfrom
guerler:remove_jquery

Conversation

@guerler
Copy link
Copy Markdown
Contributor

@guerler guerler commented Oct 10, 2025

Follow up for: #20974, Removes the History Panel proxy and associated legacy utilities

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

@guerler guerler added this to the 26.0 milestone Oct 10, 2025
@guerler guerler added area/UI-UX kind/refactoring cleanup or refactoring of existing code, no functional changes labels Oct 10, 2025
@guerler guerler marked this pull request as ready for review October 10, 2025 11:35
import {
COLLECTION_TYPE_TO_LABEL,
type CollectionBuilderType,
} from "@/components/History/adapters/buildCollectionModal";
Copy link
Copy Markdown
Member

@jmchilton jmchilton Oct 14, 2025

Choose a reason for hiding this comment

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

Certainly all these imports read much nicer!

Copy link
Copy Markdown
Member

@jmchilton jmchilton left a comment

Choose a reason for hiding this comment

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

Looks solid and green to me!

@mvdbeek mvdbeek merged commit 4af4857 into galaxyproject:dev Oct 14, 2025
32 checks passed
@guerler guerler deleted the remove_jquery branch October 14, 2025 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/UI-UX kind/refactoring cleanup or refactoring of existing code, no functional changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants