search

oppia / stale-review-request-notifier

Apache License 2.0
1 stars 2 forks source link

Fix query, tests and ci checks #3

Closed SD-13 closed 1 year ago

SD-13 commented 1 year ago

In oppia/oppia, this action failed for wrong graphql query and this PR fix that query, relevant tests, and the ci checks of this repo.

For context: This job failed because of using wrong query. Previously, the query was fetching all the discussions instead of discussions in a particular category. This PR breaks that step into two. Now, first, it is fetching the existing categories, then after filtering the category id of a particular category name, it is using another query to get the discussions in that particular category.

SD-13 commented 1 year ago

@seanlip @U8NWXD Sorry for the inconvenience, please take a look at the changes. Thanks!