Skip to content

Conversation

ajsmth
Copy link
Contributor

@ajsmth ajsmth commented Oct 14, 2021

Why

We've had to revert updates to the reload command because they access private APIs that are rejected by App Store reviews, however we can still include these commands (perhaps more useful anyways) on simulator builds only.

How

Wrapped the code in question with the TARGET_IPHONE_SIMULATOR macro so that it is stripped from production (archived) builds

Test Plan

Reload command should work on simulator builds by using the r key on your keyboard. This shouldn't be applied when a text input is focused. Should not appear in the Expo Go archived build, although I'm not quite sure how to go about verifying this (inspect with the strings command maybe?) @brentvatne your thoughts on this would be appreciated

Checklist

  • Documentation is up to date to reflect these changes (eg: https://docs.expo.dev and README.md).
  • This diff will work correctly for expo build (eg: updated @expo/xdl).
  • This diff will work correctly for expo prebuild & EAS Build (eg: updated a module plugin).

@ajsmth ajsmth requested review from brentvatne and tsapeta October 14, 2021 00:24
@ajsmth ajsmth self-assigned this Oct 14, 2021
@expo-bot expo-bot added the bot: passed checks ExpoBot has nothing to complain about label Oct 14, 2021
Copy link
Contributor

@tsapeta tsapeta left a comment

Choose a reason for hiding this comment

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

Cool 👍 Yeah, strings or otool could be helpful to test this

@brentvatne brentvatne merged commit 05a9892 into master Oct 14, 2021
@brentvatne brentvatne deleted the andrew/reload-command-simulator branch October 14, 2021 21:20
Kudo added a commit to expo/react-native that referenced this pull request Apr 26, 2022
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
Kudo added a commit to expo/react-native that referenced this pull request Apr 26, 2022
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
Kudo added a commit to expo/react-native that referenced this pull request May 13, 2022
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
douglowder pushed a commit to react-native-tvos/react-native-tvos that referenced this pull request May 23, 2022
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
Kudo added a commit to expo/react-native that referenced this pull request May 24, 2022
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
Kudo added a commit to expo/react-native that referenced this pull request Jun 23, 2022
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
Kudo added a commit to expo/react-native that referenced this pull request Jun 29, 2022
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
Kudo added a commit to expo/react-native that referenced this pull request Sep 27, 2022
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
Kudo added a commit to expo/react-native that referenced this pull request Oct 4, 2022
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
Kudo added a commit to expo/react-native that referenced this pull request Oct 13, 2022
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
Kudo added a commit to expo/react-native that referenced this pull request Jan 9, 2023
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
Kudo added a commit to expo/react-native that referenced this pull request Jan 12, 2023
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
Kudo added a commit to expo/react-native that referenced this pull request Feb 1, 2023
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
Kudo added a commit to expo/react-native that referenced this pull request May 21, 2023
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.

(cherry picked from commit 522d724)
Kudo added a commit to expo/react-native that referenced this pull request Jun 1, 2023
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
Kudo added a commit to expo/react-native that referenced this pull request Jun 21, 2023
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Oct 19, 2023
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Oct 25, 2023
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Nov 7, 2023
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Nov 13, 2023
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Nov 20, 2023
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Feb 6, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
vonovak pushed a commit to expo/react-native that referenced this pull request Feb 21, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
vonovak pushed a commit to expo/react-native that referenced this pull request Feb 21, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
vonovak pushed a commit to expo/react-native that referenced this pull request Feb 25, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
alanjhughes pushed a commit to expo/react-native that referenced this pull request Mar 7, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
alanjhughes pushed a commit to expo/react-native that referenced this pull request Mar 14, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
alanjhughes pushed a commit to expo/react-native that referenced this pull request Mar 18, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
alanjhughes pushed a commit to expo/react-native that referenced this pull request Mar 25, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Mar 26, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
alanjhughes pushed a commit to expo/react-native that referenced this pull request Apr 1, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Apr 3, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
alanjhughes pushed a commit to expo/react-native that referenced this pull request Apr 8, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
alanjhughes pushed a commit to expo/react-native that referenced this pull request Apr 15, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
alanjhughes pushed a commit to expo/react-native that referenced this pull request May 1, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
chrfalch pushed a commit to expo/react-native that referenced this pull request Jun 2, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Jun 4, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Jun 5, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Jun 9, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Jun 12, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Jun 18, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Jun 18, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Jul 1, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Jul 1, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Jul 1, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Jul 31, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Jul 31, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Aug 5, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Aug 12, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Aug 15, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
gabrieldonadel pushed a commit to expo/react-native that referenced this pull request Aug 27, 2025
RCTKeyCommands uses iOS private symbols to hook keyboard commands.
originally it's debug build only, however, we enable `RCT_DEV` in Expo Go even on release build.
this change updates the functionalities for simulator build only
which aligns to expo/expo#14735.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: passed checks ExpoBot has nothing to complain about
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants