Skip to content

Don't escape link text title#110

Merged
PVince81 merged 1 commit into
masterfrom
unescape-link-subject
Aug 22, 2017
Merged

Don't escape link text title#110
PVince81 merged 1 commit into
masterfrom
unescape-link-subject

Conversation

@PVince81

Copy link
Copy Markdown
Contributor
  1. Setup customgroups app
  2. Create custom group "test"
  3. Create a user "user1"
  4. Add user "user1" to custom group "test"
  5. Login as "user1"
  6. Check notification

Before this fix: the title was escaped and appeared with "
After this fix: no escaping, proper display

Since we're using jquery's text here it will already render as plain text and no escaping is needed.

@PVince81 PVince81 added this to the development milestone Aug 22, 2017
@PVince81 PVince81 self-assigned this Aug 22, 2017
@PVince81 PVince81 merged commit 5730cf2 into master Aug 22, 2017
@PVince81 PVince81 deleted the unescape-link-subject branch August 22, 2017 14:08
@PVince81

Copy link
Copy Markdown
Contributor Author

stable10: #111

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants