Remove href on super mentor badge

RESOLVED FIXED

Status

RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: brett, Assigned: jon)

Tracking

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

5 years ago
So the acknowledgment email you get when you receive a badge has inline html which isn't rendering for me in gmail, so imagine many users see this:

https://www.evernote.com/shard/s230/sh/42153c9d-33c1-4c93-9a37-2f5f09f7091a/1aea4cb37d18f47bbf25f734e90c5f85

As you can see above this also doesn't provide a link to the mentors profile properly.

I'd suggest simply changing this to "Congratulations! You earned the Webmaker Super Mentor badge (https://webmaker.mofostaging.net/badges/webmaker-super-mentor_. You can view it now on your profile page (link to profile).
Yeah, that sounds like a good idea
Attachment #8435818 - Flags: review?(jon)
(Assignee)

Comment 3

5 years ago
Comment on attachment 8435818 [details] [review]
https://github.com/mozilla/node-webmaker-postalservice/pull/14

Isn't the problem here that nunjucks is escaping that HTML? I see you used the safe filter though, so I'm not sure what's up with that...
Attachment #8435818 - Flags: review?(jon) → review-
(Assignee)

Comment 4

5 years ago
Created attachment 8440738 [details] [review]
https://github.com/mozilla/node-webmaker-postalservice/pull/16
Assignee: kate → jon
Status: NEW → ASSIGNED
Attachment #8440738 - Flags: review?(kate)
Attachment #8440738 - Flags: review?(kate) → review+

Updated

5 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.