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

Refactor libraries API logic into LibrariesManager #11359

Merged
merged 4 commits into from Feb 13, 2021

Conversation

davelopez
Copy link
Contributor

Overview

  • Move the controller logic to a manager class in preparation for sharing the logic with the FastAPI controller
  • Fixed a libraries API test that was randomly failing locally with This dataset is currently being used as input or output. You cannot edit metadata until the jobs have completed or you have canceled them.

Sometimes the dataset is not ready when calling the PATCH action resulting in the error:
"This dataset is currently being used as input or output. You cannot edit metadata until the jobs have completed or you have canceled them."
- Move all the logic from LibrariesController to a manager class that will be shared with the FastAPI controller
Only the basic param adaptation is done in the controller
@jmchilton jmchilton merged commit 6fe5224 into galaxyproject:dev Feb 13, 2021
@github-actions
Copy link

This PR was merged without a "kind/" label, please correct.

@jmchilton jmchilton added the kind/refactoring cleanup or refactoring of existing code, no functional changes label Feb 13, 2021
@davelopez davelopez deleted the refactor_libraries_api branch February 14, 2021 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/API area/libraries Data libraries area/testing/api area/testing kind/refactoring cleanup or refactoring of existing code, no functional changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants