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

change ms-based attribute to match depends on in assay. #240

Merged
merged 2 commits into from
Jun 21, 2023

Conversation

PozhidayevaDarya
Copy link
Contributor

The dependson item in the Mass Spec assay type did not match the attribute assigned to it.

'MS Based' was changed to "MS-based" to resolve mismatch.

The dependson item in the Mass Spec assay type did not match the attribute assigned to it. 

'MS Based' was changed to "MS-based" to resolve mismatch.
@adamjtaylor
Copy link
Contributor

Oh great spot @PozhidayevaDarya! Would you create and link and issue for this so we can track on the project board. Thank you!

Assuming tests complete I will merge tomorrow morning and redeploy to staging

@PozhidayevaDarya
Copy link
Contributor Author

Sure will do and all credit definitely to @clarisse-lau ☺️

Copy link
Contributor

@adamjtaylor adamjtaylor left a comment

Choose a reason for hiding this comment

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

LGTM 🥳

@adamjtaylor adamjtaylor merged commit 116bbc2 into main Jun 21, 2023
@adamjtaylor adamjtaylor deleted the fix-mass-spec-depends-on branch June 21, 2023 21:30
@adamjtaylor
Copy link
Contributor

Merged. Have triggered redeployment to staging which should complete in 20 mins or so. https://github.com/ncihtan/HTAN-data-curator/actions/runs/5259257034

@clarisse-lau
Copy link
Contributor

Thank you!

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.

3 participants