Touch up Badges emails
ClosedPublic
Actions

Authored by chad on Mar 2 2017, 3:21 AM.

Tags

None

Referenced Files

	F15572592: D17449.diff
	Mon, May 5, 4:44 PM

	F15566768: D17449.id41961.diff
	Thu, May 1, 11:44 PM

	F15524343: D17449.id.diff
	Mon, Apr 21, 7:31 AM

	F15521899: D17449.diff
	Sun, Apr 20, 4:19 PM

	F15515418: D17449.id41948.diff
	Fri, Apr 18, 10:18 AM

	F15502523: D17449.diff
	Mon, Apr 14, 1:41 AM

	F15479344: D17449.id41948.diff
	Apr 8 2025, 6:28 AM

	F15430775: D17449.diff
	Mar 24 2025, 8:52 AM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley

Maniphest Tasks

T12270: Unbeta Badges

Commits

rP664d9fa3ed4c: Touch up Badges emails

Summary

Ref T12270. Adds the name of the badge to the subject, fixes the double description.

Test Plan

Edit lots of badges with and without descriptions, see good emails.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad created this revision.Mar 2 2017, 3:21 AM

Herald added a subscriber: Korvin. · View Herald TranscriptMar 2 2017, 3:21 AM

Harbormaster completed remote builds in B15852: Diff 41948.Mar 2 2017, 3:22 AM

epriestley accepted this revision.Mar 2 2017, 11:57 AM

This revision is now accepted and ready to land.Mar 2 2017, 11:57 AM

Closed by commit rP664d9fa3ed4c: Touch up Badges emails (authored by chad, committed by chad). · Explain WhyMar 2 2017, 2:30 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

badges/

editor/

PhabricatorBadgesEditor.php

15 lines

Diff 41961

View Options

src/applications/badges/editor/PhabricatorBadgesEditor.php

Touch up Badges emailsClosedPublicActions