-
-
Notifications
You must be signed in to change notification settings - Fork 283
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
feat(connect-web): automatic fallback to core in popup #13031
Conversation
dea89ca
to
dce6325
Compare
6793afb
to
7423600
Compare
this can be probably merged separately? 6252066 also do we have a way how to automatically detect bundling stuff we don't want to bundle? (I know I asked this some time ago but I forgot) :/ |
@@ -137,6 +137,13 @@ export const parseConnectSettings = (input: Partial<ConnectSettings> = {}) => { | |||
settings.useCoreInPopup = input.useCoreInPopup; | |||
} | |||
|
|||
if ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
more like a note for me:
btw defaults are set only on the 'front-end' level?
a84b894
to
36ce3df
Compare
36ce3df
to
25d47c2
Compare
25d47c2
to
b15c51c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good, just some left behind console.logs
b15c51c
to
1a441e7
Compare
Why auto is not set as a default? |
To keep the default behavior the same as before, so we don't accidentally break anything. |
I'd push for making it default asap, as I doubt that anyone will change it to auto manually |
lets start experimenting with it for the next release |
Description
Implement automatic fallback for core in popup.
coreMode
auto
- automatically choose the best option based on the environmentiframe
(default) - always use iframe, same way as beforepopup
- always use popup,Init_IframeBlocked
,Init_IframeTimeout
,Transport_Missing
, …)TRANSPORT.GET_INFO
, which will triggerTransport_Missing
during init if in auto modeRelated Issue
Resolve #12488
Followup
useCoreInPopup
param