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

Adding links about GDPR compliance to the footer #681

Merged
merged 3 commits into from
Jun 20, 2018

Conversation

spacekookie
Copy link
Collaborator

@spacekookie spacekookie commented May 23, 2018

This is addressing issue #679 but not done yet. We need to figure out how to handle the translations for the newly added link. Right now no text is present (not even German/ English)

Note This PR also adds translations for #669 After further investigation, they were different translations

@spacekookie spacekookie requested a review from 1000miles May 23, 2018 13:36
@ghost ghost assigned spacekookie May 23, 2018
@ghost ghost added the in progress label May 23, 2018
@holgerd
Copy link
Member

holgerd commented Jun 1, 2018

@spacekookie Thanks for this. I do not understand: Do you need text from us? cc @Svenyo

@spacekookie
Copy link
Collaborator Author

Your original issue said to discuss translations with you. We can of course only add the German and English values ourselves, but then I'm not sure what users visiting for another language would encounter.

@Svenyo
Copy link
Collaborator

Svenyo commented Jun 4, 2018

Hi @spacekookie, please find the general workflow for translations here: https://github.com/sozialhelden/wheelmap/wiki/Add-new-translations
What's most important is that...

  1. new text strings need to be integrated in the source language (English) of the corresponding .yml file and
  2. uploaded again in Transifex.
  3. When this is done, please send me a short message,
  4. then I can start the procedure for getting the translations done with Transifex.
  5. I'll send you a message, when translations are completed and the .yml files are ready for use.

@spacekookie
Copy link
Collaborator Author

Ping @Svenyo, you can start the translation process now

@Svenyo
Copy link
Collaborator

Svenyo commented Jun 18, 2018

Hi @spacekookie, I can't find the string in Transifex. Is the updated source language (English) already uploaded?

@spacekookie
Copy link
Collaborator Author

Hey @Svenyo, I added the modified .yml files for transifex in this pull request:

From what I understand these need to be provided to transifex so we can download the translated files for wheelmap.

@Svenyo
Copy link
Collaborator

Svenyo commented Jun 18, 2018

Ah okay, so I conclude It's not uploaded in Transifex yet, right?

@spacekookie
Copy link
Collaborator Author

According to this wiki article I should just upload the .yml files with the new keys. Then:

  1. The updated resource files are to be uploaded to Transifex by the product manager

So I was under the assumption that you had to do that.

@Svenyo
Copy link
Collaborator

Svenyo commented Jun 18, 2018

Okay, good!

Copy link
Collaborator

@1000miles 1000miles left a comment

Choose a reason for hiding this comment

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

@spacekookie This PR should be good to be merged. At the moment the translation files other than the DE/EN for privacy don't really add a value but since it would be a hassle to split all translations when we do tx pull I would say we merge and deploy this PR. If further translations arrive, we could open another PR. This way we don't need to wait until all translations are done. What do you think? cc @Svenyo

@1000miles 1000miles changed the title [WIP] Adding links about GDPR compliance to the footer Adding links about GDPR compliance to the footer Jun 19, 2018
@Svenyo
Copy link
Collaborator

Svenyo commented Jun 19, 2018

Yes, good idea. Is it possible to deploy it to staging first for the review before it goes to production?

@spacekookie
Copy link
Collaborator Author

@Svenyo Alright, we're merging this PR now then and deploying to staging

@spacekookie spacekookie merged commit 63a5493 into sozialhelden:master Jun 20, 2018
@ghost ghost removed the in progress label Jun 20, 2018
@Svenyo
Copy link
Collaborator

Svenyo commented Jun 25, 2018

Dear @spacekookie, just checked on staging: For the German version everything looks fine on staging, for the English version as well. When choosing all other languages, the English wording and privacy policy content should appear, though. Please adjust!

@1000miles
Copy link
Collaborator

@Svenyo cc @spacekookie

What you currently see on staging in the footer at the moment ("Datenbestimmungen" on all pages except for English locale) is expected behavior. :) :de is the default locale in the wheelmap project while :en is the default locale for translations which explains why the "Datenschutzbestimmungen" appears on all other locale pages that don't have the translations yet.

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

Successfully merging this pull request may close these issues.

4 participants