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

Login Tout & Sign Up, GDPR style #249

Merged
merged 5 commits into from
May 17, 2018
Merged

Login Tout & Sign Up, GDPR style #249

merged 5 commits into from
May 17, 2018

Conversation

eoji
Copy link
Contributor

@eoji eoji commented May 14, 2018

what

Updating the strings from the server.
Removing Facebook copy.
Updating the disclaimer.

why

| ̄ ̄ ̄ ̄ ̄ ̄|
|          G          |
|          D          |
|         P           |
|          R          |
| _______|
(\ _ /) ||
(•ㅅ•) ||
/   づ

see

cc @dannyalright @nnekab

@eoji
Copy link
Contributor Author

eoji commented May 14, 2018

@dnywh
Copy link
Contributor

dnywh commented May 14, 2018

does this PR also affect the sign-up screen, like iOS?

@eoji
Copy link
Contributor Author

eoji commented May 14, 2018

@dannyalright also the facebook confirmation page

@eoji
Copy link
Contributor Author

eoji commented May 14, 2018

Newsletter Consent Copy

@eoji eoji changed the title Login Tout, GDPR style Login Tout & Sign Up, GDPR style May 14, 2018
@ifbarrera
Copy link

@dannyalright @eoji yep, affects all three on iOS as well. So much text 😓

@eoji
Copy link
Contributor Author

eoji commented May 14, 2018

@dannyalright added the sign up screenshot!

# Conflicts:
#	app/src/main/assets/json/server-config.json
#	app/src/main/res/values-es/strings_i18n.xml
#	app/src/main/res/values-fr/strings_i18n.xml
#	app/src/main/res/values-ja/strings_i18n.xml
Copy link

@ifbarrera ifbarrera left a comment

Choose a reason for hiding this comment

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

lgtm! 👍

@eoji
Copy link
Contributor Author

eoji commented May 17, 2018

One more screenshot @dannyalright

@eoji eoji merged commit 0c78328 into master May 17, 2018
@eoji eoji deleted the log-in-tout-gdpr-copy branch May 17, 2018 18:44
@eoji
Copy link
Contributor Author

eoji commented May 18, 2018

@eoji
Copy link
Contributor Author

eoji commented May 18, 2018

Rcureton pushed a commit that referenced this pull request Jul 31, 2018
* second pass of discovery and project refresh
Rcureton pushed a commit that referenced this pull request Jul 31, 2018
* updating the strings and login tout copy

* new strings broke my other views so i am combining PRs

* updating landscape login tout copy and updating the italian strings (why do we have them?!?!!?)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants