Stream: cds hooks/committers
Topic: docs / PR #27 Remove `redirect`, add `postMessage` proposal
Github Notifications (May 22 2017 at 19:57):
I'm a little late to this discussion but wanted to share some results of some quick prototyping I did to check browser compatibility. I found that IE 10 and 11 were not able to use
postMessagewhen when they were in separate tabs or windows. It only worked when iframed.I also tested this on the current versions of Chrome, Edge, and Safari. Using
postMessageworked in both iframed and new tab/window tests.Has anyone else done similar tests? I'm hoping I'm wrong about IE 10 & 11.
Github Notifications (May 23 2017 at 05:12):
IE 10 has the challenges you've described.
https://stackoverflow.com/questions/16226924/is-cross-origin-postmessage-broken-in-ie10
covers the crux of it.
Github Notifications (Jun 06 2017 at 12:25):
In light of the limitations of using _postMessage_ with IE 10 & 11 that would require the SMART app to be iframed, is this cause for concern? I'm not sure how much longer those browsers will be officially supported, but I know it can be difficult to get users to change their browsers, particularly when they are part of large organizations with IT departments that are in control of what's on their machines. The nice thing about the redirect url is that it's browser agnostic, though does require a server.
Github Notifications (Jan 27 2018 at 23:40):
Given that we've moved decisions out of 1.0, and considering that our specification has changed quite radically (eg, slate -> mkdocs) since this PR was opened, I'm closing this.
When we tackle decisions post 1.0, we can look at this PR for great ideas for possible implementation strategies.
Github Notifications (Jan 27 2018 at 23:40):
kpshek closed PR #27
Last updated: Apr 12 2022 at 19:14 UTC
