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

DIG-1833: Query returns all donors to the clinical data table when no donors match a genomic query #58

Merged
merged 14 commits into from
Oct 24, 2024

Conversation

OrdiNeu
Copy link
Collaborator

@OrdiNeu OrdiNeu commented Oct 24, 2024

Jira Ticket

Description

This fixes a bug where Query would return all donors authorized to a user when searching on a gene with no matching donors for the current user, but while there is a matching donor that the user is not authorized to view.

Screenshots

Screencast.from.2024-10-24.10-57-40.webm

To test

  1. make recompose-query with this branch
  2. Log in as user2
  3. Search SLX9
  4. Nothing should show up, though a donor does exist for user1
  5. Nothing else should have changed -- when no matches exist or when logged in as the correct user it should still work.

@OrdiNeu OrdiNeu requested a review from mshadbolt October 24, 2024 15:01
Copy link
Contributor

@mshadbolt mshadbolt left a comment

Choose a reason for hiding this comment

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

works for me!

Copy link
Contributor

@mshadbolt mshadbolt left a comment

Choose a reason for hiding this comment

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

Actually I just noticed that genomic completeness is now not working properly for unauthorized programs. it is only showing genomic completeness for SYNTH_02
Screenshot 2024-10-24 at 9 12 56 AM

Copy link
Contributor

@mshadbolt mshadbolt left a comment

Choose a reason for hiding this comment

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

works for completeness too after updating hotfixed htsget

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