Skip to content

Added tag to remove all nbsp instances in a string - #4999

Merged
wes-otf merged 3 commits into
mainfrom
fix/remove-nbsp-from-emails
Oct 2, 2026
Merged

wes-otf merged 3 commits into
mainfrom
fix/remove-nbsp-from-emails

Conversation

@wes-otf

@wes-otf wes-otf commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Fixes #4988. Add template tag that can be used to remove all nbsp chars from a string

Test Steps

  • Ensure no more " "s show in determination emails

@wes-otf wes-otf added Type: Bug Bugs! Things that are broken :-/ Type: Patch Mini change, used in release drafter labels Sep 22, 2026
@wes-otf
wes-otf requested a review from frjo September 22, 2026 16:55
return f"{an_or_a(text)} {text}"


@register.filter

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@register.filter(is_safe=True)
@stringfilter

Can replace the mark_safe() part.

Comment thread hypha/apply/utils/templatetags/apply_tags.py Outdated
Comment thread hypha/apply/utils/tests/test_tags.py Outdated
@wes-otf

wes-otf commented Oct 2, 2026

Copy link
Copy Markdown
Contributor Author

should be fixed! the original solution was admittedly a little overkill

@wes-otf
wes-otf requested a review from frjo October 2, 2026 16:07
@frjo frjo added Type: Enhancement This is an improvement of an existing thing (not a new thing, which would be a feature). and removed Type: Bug Bugs! Things that are broken :-/ labels Oct 2, 2026
@wes-otf
wes-otf merged commit 4d38a56 into main Oct 2, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Enhancement This is an improvement of an existing thing (not a new thing, which would be a feature). Type: Patch Mini change, used in release drafter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Handle TinyMCE inserted   in e-mails

2 participants