Skip to content
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

feat: update to ui5-webcomponents 1.20.0 #103

Merged
merged 2 commits into from
Dec 6, 2023
Merged

Conversation

ilhan007
Copy link
Member

@ilhan007 ilhan007 commented Dec 5, 2023

Updates the dependency to latest ui5-webcomponents 1.20.0

Notable changes that concerns Reactive Forms support:

- Select web component has new property "value" that should fix FormControl issues as the Select ngx wrapper relies on setting/getting value
SAP/ui5-webcomponents#7865

- TextArea and StepInput now provide @formEvents, @formProperty so that respective ngx wrappers are extending the GenericControlValueAccessor and thus support FormControl properly
SAP/ui5-webcomponents#7859
Copy link

netlify bot commented Dec 5, 2023

Deploy Preview for ui5-webcomponents-ngx ready!

Name Link
🔨 Latest commit 2cfdd57
🔍 Latest deploy log https://app.netlify.com/sites/ui5-webcomponents-ngx/deploys/657034a346080c00077e1978
😎 Deploy Preview https://deploy-preview-103--ui5-webcomponents-ngx.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@g-cheishvili
Copy link
Contributor

attachComponentStateFinalized and detachComponentStateFinalized methods are added, is that intended? Should those be exposed?

@g-cheishvili
Copy link
Contributor

Side navigation output type is broken, it contains multiple same items in the enum and it also contains undefined

@ilhan007 ilhan007 mentioned this pull request Dec 5, 2023
4 tasks
@g-cheishvili
Copy link
Contributor

Issues are fixed with #104

* fix: fixed component parsing and generation

* fix: using only public methods

* fix: updated snapshots

* chore: updated snapshots
@ilhan007 ilhan007 merged commit b458553 into main Dec 6, 2023
6 checks passed
@ilhan007 ilhan007 deleted the feat-update-to-webc-1.20.0 branch December 6, 2023 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants