Skip to content

Add prompt for users to download correct app architecture version - #15

Merged
derekblank merged 2 commits into
trunkfrom
feat/arm64-prompt
Apr 23, 2024
Merged

Add prompt for users to download correct app architecture version#15
derekblank merged 2 commits into
trunkfrom
feat/arm64-prompt

Conversation

@derekblank

Copy link
Copy Markdown
Contributor

Proposed Changes

Checks Studio architecture and prompts user to download the appropriate version of the app.

  • Adds app.runningUnderARM64Translation to app globals
  • Adds native system prompt to link user to correct app download
Screenshot 2024-04-23 at 6 54 33 pm

Testing Instructions

  • Use the app with an arm64 translated architecture (or set the value of arm64translation to true in ipc-handlers.ts.
  • Observe the prompt to download the correct version of the app

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?
@derekblank

Copy link
Copy Markdown
Contributor Author

@fluiddot These are the same changes approved via https://github.com/Automattic/local-environment/pull/288#pullrequestreview-2014492645, but in this PR the dialog copy was updated to "Mac with Apple Silicon Chip" to match the copy on the Studio Download button.

@fluiddot fluiddot 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.

LGTM 🎊 !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants