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

[23.1] Fix History item deletion/undeletion reactivity with filter and ToolForm input field #16889

Conversation

ahmedhamidawan
Copy link
Member

@ahmedhamidawan ahmedhamidawan commented Oct 19, 2023

Reactivity with History filter

If you apply the filter: deleted:any, your deleted items come into view along with other active items. But when you undelete an item, there is a bug where it is hidden even though the current filter is deleted:any

Before (Bug) After (Fixed)
history_items_store_delete_before.mp4
history_items_store_delete_after.mp4
Undeleted item goes away even though filter includes "deleted:any" Undeleted item comes back into the panel as an active item

Reactivity with FormDisplay form input

If there is a change in the current history that doesn't affect currently selected item in the Tool form, we do not reflect the change in the select field. Fix: Do Vue.set(input, "attributes", newValue); instead of just input.attributes = newValue;

Before (Bug)
If you delete/undelete an item in the history that is not already selected in the select field, the field does not react to changes.
tool_form_select_dataset_reactive_before.mp4
After (Fixed)
The field now reacts to a change in the current history
tool_form_select_dataset_reactive_after.mp4

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.

@github-actions github-actions bot added this to the 23.2 milestone Oct 19, 2023
@ahmedhamidawan ahmedhamidawan modified the milestones: 23.2, 23.1 Oct 19, 2023
@ahmedhamidawan ahmedhamidawan changed the title [23.1] Fix HistoryPanel filter reactivity with deletions/undeletions [23.1] Fix History item deletion/undeletion reactivity with filter and ToolForm input field Oct 19, 2023
@davelopez
Copy link
Contributor

Thanks for backporting the minimal fix and following up with the refactoring in #16873!

The selenium test failure seems related though 🤔

@ahmedhamidawan

This comment was marked as resolved.

@dannon dannon merged commit bdbcd72 into galaxyproject:release_23.1 Oct 24, 2023
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants