-
Notifications
You must be signed in to change notification settings - Fork 239
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
Shim ICAL to prevent using the global object #3682
Conversation
The library ical.js heavily depends on instanceof checks which will break if two separate versions of the library are used (e.g. bundled one and global one). Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
d0e6bb9
to
1cc13f4
Compare
/backport to stable2.3 |
Codecov Report
@@ Coverage Diff @@
## master #3682 +/- ##
=========================================
Coverage 27.70% 27.70%
Complexity 123 123
=========================================
Files 165 165
Lines 6018 6018
Branches 877 877
=========================================
Hits 1667 1667
Misses 4351 4351
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
Hm, my solution back then was to "fix" ICAL.js, see nextcloud/tasks@d7a24b0 and https://github.com/raimund-schluessler/ical.js/commit/5d069324d17dd4ce1e214b7c7aab9ddf7521e6b5. Might be worth checking if your definitely cleaner solution also works for Tasks. |
/backport to stable2.4 |
The backport to stable2.3 failed. Please do this backport manually. |
The backport to stable2.4 failed. Please do this backport manually. |
/backport to stable2.4 |
Fix #3649
The library ical.js heavily depends on instanceof checks which will break if two separate versions of the library are used (e.g. bundled one and global one).
Ref https://webpack.js.org/guides/shimming/
How to test