Page MenuHomePhorge

Rebrand: Add "path" entries to PlatformSymbols
ClosedPublic

Authored by avivey on Jul 10 2023, 20:42.
Tags
None
Referenced Files
F3281942: D25343.1742698279.diff
Sat, Mar 22, 02:51
F3278195: D25343.1742614890.diff
Fri, Mar 21, 03:41
F3275090: D25343.1742588693.diff
Thu, Mar 20, 20:24
F3236476: D25343.1742173264.diff
Sun, Mar 16, 01:01
F3225025: D25343.1742090216.diff
Sat, Mar 15, 01:56
F3224464: D25343.1742036669.diff
Fri, Mar 14, 11:04
F3221536: D25343.1741859936.diff
Wed, Mar 12, 09:58
F3221535: D25343.1741859933.diff
Wed, Mar 12, 09:58

Details

Summary

Ref T15006. Change a few places that were mentioning phabricator path.

Test Plan

Mk1 eyeball on each effected page.

Diff Detail

Repository
rP Phorge
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

avivey requested review of this revision.Jul 10 2023, 20:42
src/infrastructure/markup/markuprule/PhutilRemarkupEvalRule.php
71

I wonder why the original author proposed to use pht() here.

Maybe to allow message override (?)

Yeah why not. asd

Premising that if we do not use pht() we are losing string-overriding support. It's weird but maybe our glorious Evan P. wanted to override "phorge/" with → "AHAHAHAHAHALOLPHORGE/". Maybe this could be re-introduced in the future adding a function like getPlatformClientNameLocalized() and getPlatformClientPathLocalized().

This revision is now accepted and ready to land.Aug 15 2023, 07:55

I don't understand why the pht was there - it's a "path", why would it be translated?

Maybe to be it consistent with "super-rebranded-product/" :D I don't know