search

elixir-cloud-aai / proTES

Proxy service for injecting middleware into GA4GH TES requests
Apache License 2.0
5 stars 6 forks source link

fix: query filter in list task endpoint #178

Closed Ayush5120 closed 1 month ago

Ayush5120 commented 2 months ago

Fixes #177 Fixes #175

Summary by Sourcery

This pull request addresses a bug in the list tasks endpoint by ensuring that 'user_id' and 'name_prefix' filters are only added to the filter dictionary if they are not None.

sourcery-ai[bot] commented 2 months ago

Reviewer's Guide by Sourcery

This pull request addresses issue #177 by fixing the query filter in the list task endpoint. The changes ensure that the filters for 'user_id' and 'name_prefix' are only added to the filter dictionary if they are not None, preventing potential errors when these values are missing.

File-Level Changes

Files Changes
pro_tes/ga4gh/tes/task_runs.py Improved the robustness of the query filter in the list task endpoint by adding None checks for 'user_id' and 'name_prefix'.
Tips - Trigger a new Sourcery review by commenting `@sourcery-ai review` on the pull request. - You can change your review settings at any time by accessing your [dashboard](https://sourcery.ai/dashboard): - Enable or disable the Sourcery-generated pull request summary or reviewer's guide; - Change the review language; - You can always [contact us](mailto:support@sourcery.ai) if you have any questions or feedback.