Skip to content
Snippets Groups Projects

[Sprint/PinkPanther](feat): Force hashtag pipe to lowercase in discovery #2088

Merged Ben requested to merge feat/lowercase-tag-discovery-2088 into master

Closes #2088 (closed)

Summary

When clicking a hashtag from a post, this forces the querystring to take the hashtag in as lowercase. On production right now; clicking #NAME would send you to /newsfeed/global/top;hashtag=NAME;period=24h, and we want /newsfeed/global/top;hashtag=name;period=24h

Testing

Try variations on capitalising tags and clicking them, it should link through and relevant content should be found.

Merge request reports

Checking pipeline status.

Approved by

Merged by Mark HardingMark Harding 5 years ago (Oct 17, 2019 12:47pm UTC)

Merge details

  • Changes merged into master with ecb915ef (commits were squashed).
  • Deleted the source branch.
  • Auto-merge enabled

Pipeline #89485870 failed

Pipeline failed for ecb915ef on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading