Skip to content

TR_186 Add/Modify Contact Screen for PreAlpha#93

Merged
renburnett merged 1 commit into
prealpha/mainfrom
186_contact_screen
Jun 13, 2020
Merged

TR_186 Add/Modify Contact Screen for PreAlpha#93
renburnett merged 1 commit into
prealpha/mainfrom
186_contact_screen

Conversation

@renburnett

Copy link
Copy Markdown
Collaborator

[TR_186] Add/Modify Contact (Formerly Help) Screen for PreAlpha

NOTE: This PR makes use of code from #39


Please check if the PR fulfills these requirements

  • The changes don't come from your master branch. (Source)

Figma

Pre Alpha Update: TR 186

Original Help Screen: TR 127

  • The ask for the Help Screen has changed, some of the relevant code from PR 39 was used

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

  • Add a Feature (New Screen)
  • Add a Component (ContentHeader)

What is the current behavior? (You can also link to an open issue here)

  • When a user clicks on the Help link in the menu drawer, they are redirected back to the Donations Screen (their Dashboard).

What is the new behavior? (if this is a feature change)

  • The Help Screen was commented out and replaced with a Contact Us screen. Now when a user clicks the Contact Us link in the menu drawer, they are taken to a prealpha-ready Contact Page.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

  • No breaking changes that I'm aware of!

Images (before/ after screenshots, interaction GIFs, ...)


TODO:

If the user deletes their default iOS email app, a somewhat generic alert modal will popup. We'll likely want to handle this differently in future iterations.

@emarti43 emarti43 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Working on my end for iOS and Android 👍

@renburnett renburnett merged commit a079153 into prealpha/main Jun 13, 2020
@renburnett renburnett deleted the 186_contact_screen branch November 5, 2022 22:37
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.

2 participants