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

Add privacy policy link #629

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add privacy policy link #629

wants to merge 3 commits into from

Conversation

robbiemccorkell
Copy link
Contributor

Adds a link to the privacy policy on the "Support Us" page. This might help if Apple review gets angry with us:

screenshot 2018-11-19 at 8 22 06 pm

Pre-flight check-list

Before raising a pull request

  • Documentation
  • Unit tests

Pre-merge check-list

  • Link to Trello ticket/GitHub issue (If applicable)
  • Any risky areas identified
  • Test plan provided
  • Tester approved

Tester check-list

  • Tested on multiple devices / OS versions (Test on the physical device(s) you have access to, otherwise use BrowserStack):

    • Galaxy S8 (Android 7.0/8.0)
    • Galaxy S7 (Android 6.0)
    • iPhone X (iOS 11.x)
    • iPhone 8 (11.x)
    • iPhone 7 (iOS 10.x)

    Optional:

    • Google Pixel 2
    • Galaxy S8 Plus
    • Galaxy S7 Edge
    • Galaxy S6
    • iPhone 7+
    • iPhone 6

Accessibility Testing (If applicable)

  • Text-to-Voice (Android: Voice Assistant / iOS: Speak Selection)
  • Large Text (=<200%)
  • Landscape Screen orientation

Weak connection testing (If applicable)

  • Airplane mode
  • 2G/3G Network Settings

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.

2 participants