Skip to content

Fix hashtags not recognized at the beginning of the string#9616

Merged
andreslucena merged 1 commit intodecidim:developfrom
mainio:fix/hashtags-at-beginning
Sep 13, 2022
Merged

Fix hashtags not recognized at the beginning of the string#9616
andreslucena merged 1 commit intodecidim:developfrom
mainio:fix/hashtags-at-beginning

Conversation

@ahukkanen
Copy link
Copy Markdown
Contributor

🎩 What? Why?

When adding e.g. a debate without the rich text editors and the hashtag is at the very beginning of the string it is not recognized currently after #9221.

This PR fixes the issue.

📌 Related Issues

Testing

  • Make sure you don't have rich text editors enabled
  • Allow debate creation for participants
  • Create a debate with a hashtag at the beginning of the description and submit it
  • See the content for that debate from the DB where the hashtag is not correctly parsed

@ahukkanen ahukkanen added module: core type: fix PRs that implement a fix for a bug labels Jul 27, 2022
Copy link
Copy Markdown
Member

@andreslucena andreslucena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍🏽
If we keep tuning this, we should probably refactor it to the multiline format, so we can add comments, but for now it's OK for me.

@andreslucena andreslucena merged commit 2c5d638 into decidim:develop Sep 13, 2022
@ahukkanen ahukkanen deleted the fix/hashtags-at-beginning branch September 13, 2022 11:14
entantoencuanto added a commit that referenced this pull request Sep 13, 2022
* develop:
  Add missing character on code block (#9798)
  Fix hidden error messages on the registration form (#9625)
  Add documentation about configuring ActiveStorage / dynamic file uploads (#9777)
  Add documentation section about customizing cells (#9622)
  Fix hashtags not recognized at the beginning of the string (#9616)
  Fix version pages showing a HTTP 500 error when the version does not exist (#9615)
  Fix multitenant organizations stats cache (#9605)
  Prevent the account edit route through Devise (#9611)
  Fix iframe disabling producing invalid HTML (#9685)
  Fix import of images on spaces (#9779)
  Fix order of last activities (#9756)
  Fix leaking emails on admin user search controller (#9791)
  Ignore participatory spaces without models in meetings visible_for scope (#9790)
entantoencuanto added a commit that referenced this pull request Sep 15, 2022
* develop: (24 commits)
  Add develop index to the documentation (#9666)
  Fix initiatives components (#9633)
  Fix conference speaker avatars (#9643)
  Update `rokroskar/workflow-run-cleanup-action` GitHub action to v0.3.3 (#9750)
  Fix character counter for the WYSIWYG editor (#9680)
  Fix posting comments before the initial load has run (#9614)
  Fix parallel tests port in use (#9661)
  Split parallel test coverage reports into their own folders (#9686)
  Improve admin panel user experience regarding title links and order of actions (#9496)
  Fix title and description too long in initiatives spec sometimes (#9648)
  Fix API GraphiQL system spec with newer ChromeDriver (#9642)
  Add missing character on code block (#9798)
  Fix hidden error messages on the registration form (#9625)
  Add documentation about configuring ActiveStorage / dynamic file uploads (#9777)
  Add documentation section about customizing cells (#9622)
  Fix hashtags not recognized at the beginning of the string (#9616)
  Fix version pages showing a HTTP 500 error when the version does not exist (#9615)
  Fix multitenant organizations stats cache (#9605)
  Prevent the account edit route through Devise (#9611)
  Fix iframe disabling producing invalid HTML (#9685)
  ...
eliegaboriau pushed a commit to eliegaboriau/decidim that referenced this pull request Oct 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: core type: fix PRs that implement a fix for a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants