Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • Sign in / Register
Minds Frontend
Minds Frontend
  • Project overview
  • Repository
  • Issues 399
  • Merge Requests 65
  • CI / CD
  • Security & Compliance
  • Packages
  • Analytics
  • Wiki
  • Snippets
  • Members
  • Collapse sidebar
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Minds
  • Minds FrontendMinds Frontend
  • Merge Requests
  • !772

Open
Opened 4 days ago by Ben Hayward@benhayward.ben
Report abuse

Fix/url hashtag count #2361

  • Overview 1
  • Commits 9
  • Pipelines 2
  • Changes 9

Summary

Closes #2361

** note this MR includes the new installer changes, wait for that before merge **

Added in a check for whether the matching tag is located in a URL.

Steps to Test

This one is a bit more "get in there and try your best to break it" in terms of testing, but that said, to get a foot in the door:

  1. Go on the site, make a post with 5 hashtags; try to trip the Regex up and make some invalid tags to see how it responds to the max count.
  2. Try a URL with a hashtag in.
  3. Check the hashtags links go through to the correct place.
  4. Try to use the hashtag selector to bypass the limit
  5. Ensure when you add a tag it shows up in the selector

Estimated Regression Scope

This change could affect the poster in different environments, it could perhaps trigger something that should still not be a hashtag as a hashtag

Request to merge fix/url-hashtag-count into master
The source branch is 4 commits behind the target branch
Open in Web IDE
Pipeline #117301195 passed with warnings for 547ff898 on fix/url-hashtag-count
              Requires 3 more approvals from Devs, Deployers, and QA.
              Brian Hatchet Marcelo Rivera Rami Albatal Xander Miller Emiliano Balbuena
              Ready to be merged automatically. Ask someone with write access to this repository to merge this request

              Closes #2361

              Deletes source branch

              • Ben Hayward @benhayward.ben added Squad::Yellow scoped label 4 days ago

                added scoped label

              • Ben Hayward @benhayward.ben added 12 commits 5 hours ago

                added 12 commits

                • 0e69ebb8...745769a5 - 11 commits from branch master
                • 547ff898 - Merge branch 'master' of gitlab.com:minds/front into fix/url-hashtag-count

                Compare with previous version

              • Emiliano Balbuena
                Emiliano Balbuena @edgebal · 36 minutes ago
                Developer

                Works pretty good for most of the cases I tried, but it seems that it parses the first half of a word that starts with the pound sign as a hashtag as well. Check the image below:

                image

                Edited by Emiliano Balbuena 36 minutes ago
              • Emiliano Balbuena @edgebal added Status::Requires Changes scoped label 36 minutes ago

                added scoped label

              • Emiliano Balbuena @edgebal assigned to @benhayward.ben 36 minutes ago

                assigned to @benhayward.ben

              Please register or sign in to reply
              Assignee
              Ben Hayward's avatar
              Ben Hayward @benhayward.ben
              None
              Milestone
              None
              Time tracking
              No estimate or time spent
              2
              Labels
              Squad::Yellow Status::Requires Changes
              Lock merge request
              Unlocked
              10
              10 participants
              user avatar
              user avatar
              user avatar
              user avatar
              user avatar
              user avatar
              user avatar
              Reference: minds/front!772