RocketChat / RocketChat/Rocket.Chat.js.SDK
Bug: Generator functions result in a syntax error in IE11
@tassoevan is already working on this.
Since Sep 17, 2019.
- Dominant language
- TypeScript
- Stars
- 146
- Forks
- 99
- Avg merge
- 7h 59m
- Merged PRs (30d)
- 17
Description
In regards to RocketChat/Rocket.Chat.Livechat#180 and a lookup at the current definition at https://github.com/RocketChat/Rocket.Chat.Livechat/blob/dev/package.json#L111
It seems as it is at least intended to support IE11, as far as the current browserlist definition goes.
Since the update to the latest Rocket.Chat Version, which features this new Widget, some customers of maintained sites are not able to use the Chat functionality because it is broken in IE11.
So will there be an official statement in the near future, if IE11 will be supported or not?
Maybe this problem could be solved by minor changes to the codebase and/or babel preset?
I would also like to see IE11 die already, but reality is that many institutions, especially rural ones, in many countries will have and use the IE11 shipped with their operating system or as ruled by their administrators or policies. So support for the latest IE version (11) should be mandatory for widespread acceptance/usage of this software.
Also to mention, that the old widget used to just worked with IE11 and the new one does not, which may prevent some from upgrading to the latest version of Rocket.Chat because support for that old browser is missing within the shipped Livechat widget.
SCRIPT1005: Exptected '('
Rocket.Chat.js.SDK.chunk.27bc4.js (1,2193)
SCRIPT1005: Expected '('
vendor.chunk.8bd3b.js (1,255383)
The related lines feature a generator function. I guess IE11 does not have Generator functions and we would need to Polyfill them through babel e.g. @babel/plugin-transform-regenerator
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.