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

Subject in mail is Filesender when using dutch language in browser #417

Closed
janwout opened this issue Sep 11, 2018 · 3 comments
Closed

Subject in mail is Filesender when using dutch language in browser #417

janwout opened this issue Sep 11, 2018 · 3 comments

Comments

@janwout
Copy link

janwout commented Sep 11, 2018

Also in German. When you send a file with filesender 2.1, with a browser with dutch or german language, the subject you enter is ignored and Filesender is the subject of the email sent to the receiver.

@monkeyiq
Copy link
Contributor

I have added some information to the i18n page about this
https://docs.filesender.org/v2.0/i18n/

basically, the system uses the "subject: " key in the translation file as a specific thing to change for email subjects. The dutch language file was translating that to "onderwerp: " which was then not being seen by FileSender as something it should translate and use as the email subject line. It is a little confusing to have this special english keyword that must remain english in the translation on poeditor, but that is how the system runs at the moment. It simplifies the code not to try to look for a language translation of "subject:" but just for the english keyword for this purpose. I have updated the poeditor translation for this email and will be importing updated translations from poeditor to github shortly.

@monkeyiq
Copy link
Contributor

I see that #476 changes a bunch of the onderwerp: to subject: too. I have just rechecked the following list which appear to already have been updated. I imagine there is a history somewhere in poeditor though it isn't jumping out at me. I think I will email the filesender-dev list to mention this to other folks.

bounce_report
daily_summary.mail.php
download_complete.mail.php
email_feedback.mail.php
file_deleted.mail.php
files_downloaded.mail.php
guest_access_upload_page.mail.php
guest_cancelled.mail.php
guest_created.mail.php
guest_created_receipt.mail.php
guest_expired.mail.php
guest_reminder.mail.php
guest_upload_complete.mail.php
guest_upload_start.mail.php
recipient_deleted.mail.php
recipient_feedback.mail.php
report_attached.mail.php
report_inline.mail.php
storage_usage_warning.mail.php
transfer_autoreminder_receipt.mail.php
transfer_available.mail.php
transfer_deleted.mail.php
transfer_deleted_receipt.mail.php
transfer_expired.mail.php
transfer_expired_receipt.mail.php
transfer_reminder.mail.php
translate_email_footer.mail.php
upload_complete.mail.php
transfer_deleted_receipt
transfer_expired_receipt
guest_upload_complete
guest_upload_complete_confirmation_to_guest
upload_complete
guest_upload_start
guest_created
guest_created_receipt
guest_reminder
guest_cancelled
guest_expired
transfer_available

@monkeyiq
Copy link
Contributor

Thank you for reporting this. I have emailed the filesender-dev to widen the group of people who are aware of this issue. The Dutch files above all seem in order and will be imported from poeditor before the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants