-
Notifications
You must be signed in to change notification settings - Fork 100
[USH-1699] Identify unlocalized strings in the web client and add them to en.json #1494
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
Conversation
…icker, default settings header
…d polling service
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.
Well done! Approved!
{ displayNane: this.translate.instant('activity.categories'), value: 'tags' }, | ||
{ displayNane: this.translate.instant('activity.survey'), value: 'form' }, | ||
{ displayNane: this.translate.instant('activity.status'), value: 'status' }, | ||
]; |
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.
Is it actually called displayNane? :D I see it was called that before too, maybe we should change it to displayName, or am I missing something important?
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.
Ah yes, I should change that. I almost thought it was named like that on purpose but I can still change it :)
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.
Actually, when thinking about it, its for the activity-view that is most likely going to change very soon, so lets leave it as it. Its out of scope for this ticket :)
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.
Aah ok, actually I have already made the changes, but I think we can leave it at that
…m to en.json (#1494) * Translations - Breadcrumbs * Translations - Login/Authentication * Translations: Post timelines, posts loading message, date shows up here text * Reverting timeline translations for now * Translation: OK Button, Uploading, Activty View * Translation: Data source subheading * Slight adjustment to solve 'Add webhook' json file conflicts * Translations: Survey & General Confirm Modal, snackbar, Survey Colorpicker, default settings header * Translations: Activity page * Translations: Adding more strings into en.json for map view options * Translations: Onboarding modals * Translations: Sources Filters * Translations: Timelines for post metadata and post conversations * Just a slight fix * Translations: showError() in survey-item, user-settings components and polling service * Translations: Close button in all snackbars * Translations: Location Select strings * Translations: Image uploader special characters text * Translations: Post edit component * Translations: Post not authorized to edit * Translations: Remaining strings * Renamed 'displayNane' to 'displayName'
No description provided.