Skip to content

London | 26-SDC-Mar | Beko | Sprint 1 | Hashtag link doesn't work - b…#204

Open
Abubakar-Meigag wants to merge 2 commits into
CodeYourFuture:mainfrom
Abubakar-Meigag:beko-hashtag-link-doesn't-work-bug
Open

London | 26-SDC-Mar | Beko | Sprint 1 | Hashtag link doesn't work - b…#204
Abubakar-Meigag wants to merge 2 commits into
CodeYourFuture:mainfrom
Abubakar-Meigag:beko-hashtag-link-doesn't-work-bug

Conversation

@Abubakar-Meigag

Copy link
Copy Markdown

…ug report

Self checklist

  • I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
  • My changes meet the requirements of the task
  • I have tested my changes
  • My changes follow the style guide

Bug report:

  • When the user clicks the hashtag, the hashtagView calls apiService unconditionally on every render and that causes an infinite refetch.

To fix it:
To prevent infinite fetch loops in hashtagView, I added a condition to skip fetch when the hashtag is unchanged

@Abubakar-Meigag Abubakar-Meigag added 📅 Sprint 1 Assigned during Sprint 1 of this module Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. Module-Legacy-Code The name of the module. labels Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Module-Legacy-Code The name of the module. Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. 📅 Sprint 1 Assigned during Sprint 1 of this module

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

1 participant