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

Give modal with instructions to install desktop client #237

Closed
quinthar opened this issue Aug 19, 2020 · 2 comments · Fixed by #238
Closed

Give modal with instructions to install desktop client #237

quinthar opened this issue Aug 19, 2020 · 2 comments · Fixed by #238
Assignees

Comments

@quinthar
Copy link
Contributor

quinthar commented Aug 19, 2020

Right now when you go to install the desktop client, you get an error message and it's very unclear how to get around it. I expect like 90% of employees who try will get this error, and not know how to fix it. Let's just anticipate that and when you click "Desktop" we show a message:

To install:

  1. Download this link
  2. Open it
  3. If you get a "cannot be opened because the developer cannot be verified" error:
    1. Hit Cancel
    2. Go to System Preferences > Privacy > General
    3. Click Open Anyway at the bottom
    4. When it re-prompts you, click Open

Assigning MED because nearly everyone who tries to use the desktop client will experience this

@quinthar quinthar added this to the All Hand's Launch milestone Aug 19, 2020
@AndrewGable
Copy link
Contributor

It looks like modals aren't supported on react-native-web yet: necolas/react-native-web#1020

But alert() is supported:
Screen Shot 2020-08-18 at 5 33 00 PM
Screen Shot 2020-08-18 at 5 32 53 PM

While it doesn't look great, it gets the point across. Does that sound OK?

@AndrewGable AndrewGable self-assigned this Aug 19, 2020
@quinthar
Copy link
Contributor Author

Yep, better than nothing

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 a pull request may close this issue.

2 participants