-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Spanish Translations: FileUploader, Error Modal, Share Modal #1561
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
…leModal # Conflicts: # translations/locales/en-US/translations.json # translations/locales/es-419/translations.json
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.
Please check the comments for changes.
If you would like to upload more, please remove the ones you aren't using anymore by | ||
in your ` | ||
} | ||
{this.props.t('UploadFileModal.SizeLimitError')} |
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.
UploadFileModal.SizeLimitError
interpolates limitText
but it's not provided as a parameter to t()
"UploadFileModal": { | ||
"Title": "Upload File", | ||
"CloseButtonARIA": "Close upload file modal", | ||
"SizeLimitError": "Error: You cannot upload any more files. You have reached the total size limit of ${limitText}.\n If you would like to upload more, please remove the ones you aren't using anymore by\n in your " |
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 is the incorrect way to interpolate limitText
into this message.
I have verified that this pull request:
npm run lint
)It provides the translation for uploading files, error modals and share modal.