Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update more explore card sub headings to include wikipedia lang #2453

Merged
merged 5 commits into from
Jul 20, 2018

Conversation

montehurd
Copy link
Contributor

@montehurd montehurd commented Jul 19, 2018

https://phabricator.wikimedia.org/T200040

  • updates on this day explore card sub heading to include wikipedia lang:

42966559-6c9a6e18-8b52-11e8-865b-9e1afeb7f1fb

  • adds "from #lang# Wikipedia" to places feed cards

screen shot 2018-07-19 at 4 22 27 pm

NSString *language = [[NSLocale currentLocale] wmf_localizedLanguageNameForCode:self.siteURL.wmf_language];
if (language) {
return
[NSString localizedStringWithFormat:WMFLocalizedStringWithDefaultValue(@"on-this-day-sub-title-for-date-from-language-wikipedia", nil, nil, @"%1$@ from %2$@ Wikipedia", @"Subtext beneath the 'On this day' header when describing the date and which specific Wikipedia. %1$@ will be substituted with the date. %2$@ will be replaced with the language - for example, 'June 8th on English Wikipedia'"), [[NSDateFormatter wmf_utcMonthNameDayOfMonthNumberDateFormatter] stringFromDate:self.midnightUTCDate], language];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should the example be 'June 8th from English Wikipedia'? 🤓

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah good catch... fixing...

@montehurd montehurd force-pushed the update/on-this-day-card-sub-heading/T200040 branch from b0888ef to 8bb26e0 Compare July 19, 2018 20:54
@montehurd montehurd changed the title update on this day explore card sub heading to include wikipedia lang update more explore card sub headings to include wikipedia lang Jul 19, 2018
@joewalsh joewalsh merged commit 9e47dc3 into develop Jul 20, 2018
@joewalsh joewalsh deleted the update/on-this-day-card-sub-heading/T200040 branch July 20, 2018 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants