Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Removing re-fetch of categories if there is an error from the API #1495

Merged
merged 3 commits into from
Dec 10, 2024

Conversation

Angamanga
Copy link
Contributor

Removing refetch of Categories in filters if the original request fails (this caused a never ending fetch of categories in the web-client when the API returned an error).
To test (check both as logged out and logged in):

  • Go to the deployment
  • Check the network tab
  • Only one request for categories should be made
  • If the request has returned an error (happens right now in mzima-dev), no categories show up in the filters-bar (there are no Categories to show)
  • If the request has returned an error, the client should not keep trying

Copy link

linear bot commented Dec 2, 2024

@Angamanga Angamanga requested review from Ifycode and Mh-Asmi December 2, 2024 13:56
@Angamanga Angamanga marked this pull request as ready for review December 2, 2024 13:57
@Angamanga Angamanga merged commit 74cbf81 into development Dec 10, 2024
6 of 8 checks passed
tuxpiper added a commit that referenced this pull request Dec 18, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants