-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
App broken on iOS Safari 9 due to SyntaxError #11064
Comments
1.10.2 seems to have this issue on old versions of Safari in general -- even a clean install exhibits the same problem. |
Hi, do you have an easy way to test in old Safari? Maybe this core-js upgrade was the cause: Also have you tried to use |
We have been using BrowserStack for testing, which is great: We do have |
@filipenevola if you are on a Mac, you can install Xcode and launch the Simulator. From there, you can install different iOS versions and run them on various virtual Apple devices. You can even use desktop Safari's debugger. |
+1 I get a similar error in Safari 8... To reproduce, run the command "meteor create simple-todos" then run the app. Goto browserstack and use safari 8 and go to the localhost server at the meteor port number in use (http://localhost:3000). The console will show 45 errors: |
Hi, I am experiencing the same error with Meteor 1.10.2 and safari 9.1.2. Any workaround? Thanks, |
I downgraded Meteor 1.8.1 to fix the error. Still have the error with 1.8.2 & 1.8.3 |
I'm late to the party, but I'll +1. Upgrading to 1.10.2 breaks us on iOS 9. |
It's not just on IOS, tested on Mac Os today with Safari 9.1.2. |
Hi, could you test with Meteor 1.10.3-beta.5? meteor update --release 1.10.3-beta.5. thanks. |
Still does not work on 1.10.3-beta.4 -- I created a clean app and used Browserstack to test on Safari 7.1 and got errors similar to @brettg2 above. |
Hi @brianlukoff did you on beta.5? BTW beta.4 was broken due to version constraints at least if you use Please let me know! |
Just retested on beta.5 with the same result -- I am creating a clean app with |
- replaces the user to run Browserstack
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Not stale AFAIK |
Not stale |
Hey, this is not stale indeed, we are working on this. We know that this broke the first time here https://github.com/meteor/meteor/blame/830b8a40cb5df665ef9688bd7ac6cece3b1f4223/tools/isobuild/linker.js#L688 but reverting this change breaks the minifier when using dynamic imports. We are analyzing if upgrading terser or using a different approach would fix the issue but no luck so far. As @renanccastro is working on many changes on related code he is going to check this in the tree-shaking branch #11164 @denihs has added back the BrowserStack #11166 test to our CircleCI so we are going to run it again against every PR to avoid unintentional breaking changes again in the future. This test was removed a long time ago but we think this is important and then we are adding it back. For now we are going to merge this BrowserStack test using Safari 11 as minimum version but we want to reduce this version to 9 again, if we are able to fix the issue on tree-shaking branch. We decided to merge using Safari 11 now to at least avoid more breaking changes in the near future. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Not an Apple user, is Safari 9 still a thing at this point? @renanccastro |
I think the statement in the issue description still stands:
...of course it's up for discussion whether that's reason enough, considering the minuscule market share of old iOS versions, but since it used to work... 🤷 |
This is an old issue. So from an MDG biz decision, not acting on this for this long I think has in many ways obviated the need to fix it. Existing Meteor apps have had to find a workaround or punt. This was a very hard choice for us to tell our users we cannot support them anymore. When new Cordova breaks old native iOS/android builds, we have usually had the good fortune to tell customers to use a BROWSER on their old devices. My request to MDG is to do their best to test and support old browsers |
@ericoe since this issue happened we had upgraded our Browserstack tests, reactivated them and we are running them before any releases so new regressions are not going to come up, at least not as a surprise. |
I know it's iOS Safari 9, but our app used to work on this browser with Meteor
1.8.2
. I think supporting old iOS versions is important, since lots of people have old iPads laying around that they use for browsing the web. They can't update the browser past the OS version and Apple doesn't release updates to those old, yet functional devices.We are updating our app to the latest Meteor
1.10.2
and now the app is broken on iOS Safari 9.Related to unresolved & closed issue: #10957
The error message:
SyntaxError: Invalid parameters or function name in strict mode
The text was updated successfully, but these errors were encountered: