-
Notifications
You must be signed in to change notification settings - Fork 65
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
Localize Sample App #19
Conversation
- able to switch culture via ugly button - localized some of the hardcoded content into resx file - able to fetch items based on culture from Delivery API
- added language constraint - pushed hardcoded strings into resource file - modified design of language switcher based on EMS dancing goat
in general, the code lgtm however, it doesn't work because there is no localized data in the sample project (this needs to be fixed before merging this PR) |
|
||
public Task<DeliveryItemResponse<T>> GetItemAsync<T>(string codename, IEnumerable<IQueryParameter> parameters = null) | ||
{ | ||
return parameters != null ? _client.GetItemAsync<T>(codename, new[] { new LanguageParameter(_language) }.Concat(parameters)) : |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Value cannot be null.
Parameter name: second
can be seen on the homepage
--> localized data is missing in the sample project 975bf280-fd91-488c-994c-2f04416e5ee3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was caused by missing data in sample project. Should be fixed in next commit
Added localization support for sample site.