Ref T12270. This converts Badges to modular transactions for editing and awarding.
Details
- Reviewers
epriestley - Maniphest Tasks
- T12270: Unbeta Badges
- Commits
- rP3eae9a368de0: Modular Transactions for Badges
Add Badge, edit badge, award and revoke... Still going to test this some more but feel free to comment on anything obviously wrong?
Diff Detail
- Repository
- rP Phabricator
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Nice!
src/applications/badges/xaction/PhabricatorBadgesBadgeFlavorTransaction.php | ||
---|---|---|
17–23 | I think you can get rid of this completely now since Badges are on EditEngine, should be handled automatically now. | |
src/applications/badges/xaction/PhabricatorBadgesBadgeIconTransaction.php | ||
17–23 | As above, likely obsolete. | |
src/applications/badges/xaction/PhabricatorBadgesBadgeQualityTransaction.php | ||
17–23 | As above, likely obsolete. | |
32–45 | These should maaaaaybe use renderValue(...) to pick up the italic style? ModularTransactions generally uses renderValue() for titles, etc. Not sure if we want to count this sort of text as user-provided text or not. | |
src/applications/badges/xaction/PhabricatorBadgesBadgeStatusTransaction.php | ||
28 | Does this need a feed title? | |
src/applications/badges/xaction/PhabricatorBadgesBadgeTransactionType.php | ||
7–9 | Get rid of this, specific to Calendar. |
A lot of stuff just worked, so I'm going to re-test everything again because... it seemed too easy?
src/applications/badges/xaction/PhabricatorBadgesBadgeQualityTransaction.php | ||
---|---|---|
32–45 | I couldn't get that to work? The value is a number. |
src/applications/badges/xaction/PhabricatorBadgesBadgeQualityTransaction.php | ||
---|---|---|
32–45 | Oh, sorry, I mean like this? Or did that not work either? $new = $this->getNewValue(); // Raw number, like 3. $new = $this->getQualityLabel($new); // Now a string, like "Rare". $new = $this->renderValue($new); // Now a fancy string, like //Rare//. |