Replies: 1 comment
-
Hey @jhen0409 👋 This has been addressed in #1226 which was released with 4.7.0 of the devtools. The message now contains a As such I'm going to go ahead and close this discussion. Thanks for reporting this to make sure I addressed this in that PR! |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
https://github.com/apollographql/apollo-client-devtools/blob/main/src/extension/constants.ts
The message type is used common words with no any specific prefix, I think it's possible to break some apps.
Here is what I log
window.addEventListener("message", ...)
on a webpage:Could we accept a PR about add prefix like
ac-devtools:
?Background:
I'm trying to upgrade apollo-client-devtools for react-native-debugger (for old remote debugger), I need to build another bridge for WebWorker to support it. However, the current message type is not easy to identify where it comes from, and it is not easy to maintain.
UPDATE: PR is here - #1037
Beta Was this translation helpful? Give feedback.
All reactions