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

Issue 497 Android policy compliance #506

Merged
merged 6 commits into from
Feb 3, 2024

Conversation

coryzoon
Copy link
Contributor

@coryzoon coryzoon commented Jan 30, 2024

Fixes #497

Designs: https://www.figma.com/file/QC931orjhCdGoQogVg7iDZ/Mobile-App?type=design&node-id=237-1158&mode=design&t=mYaV3cg9TSzbCigN-0

Summary: Updates to the side drawer menu and settings page to support external links for account deletion and privacy policy.

  • Updated font to Karla.
  • Added library react-native-toggle-element to support custom toggle.
  • Updated side drawer menu to match designs, adding "Privacy", "Contact us", "Version", remove social icons, etc...
  • Updated settings screen to match designs, show "Zooniverse account information" & "Delete my account" when the user is logged in, updated text in some places, and updated toggle switches.
  • Updated settingsComponents.js file to update the components that support the settings screen.
  • Made side drawer menu wider to support the longer titles.

@coryzoon coryzoon requested a review from mcbouslog January 30, 2024 13:27
@@ -0,0 +1,7 @@
module.exports = {
Copy link

Choose a reason for hiding this comment

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

'module' is not defined no-undef

Switch,
View
} from 'react-native'
import React from 'react';
Copy link

Choose a reason for hiding this comment

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

Parsing error: 'import' and 'export' may appear only with 'sourceType: module'

@mcbouslog mcbouslog self-assigned this Jan 31, 2024
Copy link
Contributor

@mcbouslog mcbouslog left a comment

Choose a reason for hiding this comment

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

Looking good to me! Tested locally and confirmed UX is as expected.

Non-blocking note - I noticed in About the Zooniverse and Publications in the upper left there's a carrot I can press to go back, but in Settings and Notifications there's the Zooniverse icon that doesn't do anything when I press it. Just noting, disregard if intentional, not blocking if something to revisit subsequent to this PR.

@coryzoon coryzoon merged commit 5358a74 into master Feb 3, 2024
1 check passed
@coryzoon coryzoon deleted the issue-497-android-policy-compliance branch February 8, 2024 11:30
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.

Android policy compliance improvements: links to account deletion, privacy policy
2 participants