Paths

Table of Contentst

Differential D11813

Projects - fix translation strings in watcher edge class
ClosedPublic
Actions

Authored by btrahan on Feb 18 2015, 11:38 PM.

Tags

None

Referenced Files

	F14598126: D11813.id28478.diff
	Wed, Jan 8, 9:41 PM

	Unknown Object (File)
	Sun, Dec 29, 10:13 PM

	Unknown Object (File)
	Mon, Dec 16, 1:53 PM

	Unknown Object (File)
	Wed, Dec 11, 10:36 AM

	Unknown Object (File)
	Tue, Dec 10, 11:29 PM

	Unknown Object (File)
	Tue, Dec 10, 9:45 PM

	Unknown Object (File)
	Tue, Dec 10, 2:54 PM

	Unknown Object (File)
	Dec 9 2024, 5:31 PM

Subscribers

Details

Reviewers

Maniphest Tasks

T7319: '1 watcher(s)' as well as 'a member' in project edit logs

Commits

Restricted Diffusion Commit
rPa77127ab63bf: Projects - fix translation strings in watcher edge class

Summary

Fixes T7319. These need a "%s" for the count where they had a "%d"

Test Plan

plan in D11812 is no longer a lie! (watcher added / removed strings render correctly)

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

btrahan updated this revision to Diff 28474.Feb 18 2015, 11:38 PM

btrahan added a task: T7319: '1 watcher(s)' as well as 'a member' in project edit logs.

btrahan retitled this revision from to Projects - fix translation strings in edge class.

btrahan updated this object.

btrahan edited the test plan for this revision. (Show Details)

btrahan added a reviewer: epriestley.

Herald added subscribers: epriestley, Korvin. · View Herald TranscriptFeb 18 2015, 11:38 PM

btrahan retitled this revision from Projects - fix translation strings in edge class to Projects - fix translation strings in watcher edge class.Feb 18 2015, 11:38 PM

The %s is correct, so maybe the other ones should flip?

This is kind of unintuitive, but basically the rule is "%s" to render as "2,345,678" (i.e., with commas and periods) and "%d" to render as "2345678". So "You see %s things(s)", but "Task ID %d".

flip it

epriestley accepted this revision.Feb 18 2015, 11:44 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Feb 18 2015, 11:44 PM

btrahan updated this object.Feb 18 2015, 11:44 PM

btrahan edited edge metadata.

Closed by commit rPa77127ab63bf: Projects - fix translation strings in watcher edge class (authored by btrahan, committed by btrahan). · Explain WhyFeb 18 2015, 11:45 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

infrastructure/

internationalization/

translation/

PhabricatorUSEnglishTranslation.php

6 lines

Diff 28478

src/infrastructure/internationalization/translation/PhabricatorUSEnglishTranslation.php

Loading...