Closed Bug 922729 Opened 12 years ago Closed 12 years ago

String "Mozillian for %(mozillian_years)s years" needs pluralization

Categories

(Participation Infrastructure :: Phonebook, defect)

Product:
Participation Infrastructure
Component:
Phonebook
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
2013-10-10

People

(Reporter: JasnaPaka, Assigned: tbharath)

Details

(Whiteboard: [kb=1142465] [mentor=giorgos@mozilla.com][good first bug])

mozillians/phonebook/templates/phonebook/profile.html:95 String: "Mozillian for %(mozillian_years)s years" This string needs pluralization.
Thanks for reporting Pavel! Interested into fixing this? I can guide you through if needed.
Whiteboard: [mentor=giorgos@mozilla.com][good first bug]
I am interested in solving this bug, and I have also sent pr https://github.com/mozilla/mozillians/pull/653
Assignee: nobody → bharath_ves
Whiteboard: [mentor=giorgos@mozilla.com][good first bug] → [kb=1142465] [mentor=giorgos@mozilla.com][good first bug]
Commits pushed to master at https://github.com/mozilla/mozillians https://github.com/mozilla/mozillians/commit/c5223a90febd8528fa73cc22b42c8663794b7a60 [Fix Bug 922729] Added pluralization for the string 'Mozillian for %(mozillian_years)s years' https://github.com/mozilla/mozillians/commit/12f01c6255a1b2ef2ce7751d943bb225392c7e2b Merge pull request #657 from bharaththiruveedula/Bug922729 [Fix Bug 922729] Added pluralization for the string 'Mozillian for %(moz...
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Verified on stage. Setting my get involved field to October 2012 shows up in my profile as 'Mozillian for 1 year'. Setting it earlier than that show 'Mozillian for 2 years'. Nice attention to detail in this enhancement. Thanks for fixing this, tbharath!
Status: RESOLVED → VERIFIED
Target Milestone: --- → 2013-10-10
You need to log in before you can comment on or make changes to this bug.