Skip to content

WestMidlands| SDC NOV 2025| Sara Tahir| Sprint 1| Feature HashtagLink#109

Open
SaraTahir28 wants to merge 1 commit intoCodeYourFuture:mainfrom
SaraTahir28:feature/fixhashtaglink
Open

WestMidlands| SDC NOV 2025| Sara Tahir| Sprint 1| Feature HashtagLink#109
SaraTahir28 wants to merge 1 commit intoCodeYourFuture:mainfrom
SaraTahir28:feature/fixhashtaglink

Conversation

@SaraTahir28
Copy link

Learners, PR Template

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

Changelist

  1. Updated the formatHashtags regex so it only matches valid hashtag characters (letters, numbers, underscores) instead of everything up to the next #.

2.This prevents cases like #SwizBiz love!! from being treated as one long hashtag and generating a broken link.

3.Now hashtags stop at spaces/punctuation, so links like /hashtag/SwizBiz work correctly and show the expected blooms.

Questions

I have no Questions. Thankyou.

@SaraTahir28 SaraTahir28 added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Feb 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed.

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

1 participant