-
Notifications
You must be signed in to change notification settings - Fork 273
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
[OpenUI5Support]: Version 1.120 Fatal Issue due to Formatting function being renamed #7871
Comments
I'm having trouble right now to get the sandbox to reproduce the issue because openui5 script is not downloading or executing - I'm not sure what i did wrong - but someone could fix that. https://codesandbox.io/s/ui5-webcomponents-forked-5kchxn?file=/index.html |
It seems that your sample works with 1.120.0, but if you change the bootstrap to nightly like here, it does break. Working on a fix right now. |
Hi @scameron we plan to downport this fix for 1.19 as 1.19.1 Do you need this for older versions? |
I don't think we're waiting for this fix on our side. Did you mean to tag @i832513 instead? |
Hello Scott @i832513 we plan to downport this fix for 1.19 as 1.19.1 Do you need this for older versions? |
@ilhan007 , |
Ok, @i832513 let me know if you would be able to upgrade or rather stay on 1.15 |
@ilhan007 , |
Bug Description
Please refer to the following links:
The function
Formatting.setLegacyDateCalendarCustomizing
is renamed in version 1.120 - it is unclear at this time if OpenUI5 will deliver a new version of OpenUI5 which provides the backwards compatibility necessary to prevent fatal page errors.Affected Component
OpenUI5Support
Expected Behaviour
The OpenUI5Support static import should not crash the page no matter what version of OpenUI5 is being used.
Isolated Example
https://codesandbox.io/s/ui5-webcomponents-forked-5kchxn?file=/index.html
Steps to Reproduce
This will break the page and not load any web components.
Log Output, Stack Trace or Screenshots
OpenUI5Support.ts:140 Uncaught TypeError: i.getLegacyDateCalendarCustomizing is not a function
at Function.value (OpenUI5Support.ts:140:48)
at InitialConfiguration.ts:187:39
at w (InitialConfiguration.ts:203:2)
at l (InitialConfiguration.ts:46:2)
at l (Theme.ts:16:14)
at d (Theme.ts:71:23)
at 292 (nav-back.js:5:18)
at webpack_require (bootstrap:19:22)
at fetchLocalizedContent.js:173:1
at index.tsx:29:3
Priority
High
UI5 Web Components Version
1.19
Browser
Chrome, Edge, Firefox, Safari
Operating System
Any
Additional Context
Compatibility with version 1.120.0
Organization
SuccessFactors
Declaration
The text was updated successfully, but these errors were encountered: