Skip to content

Conversation

@Thodd
Copy link
Collaborator

@Thodd Thodd commented Aug 12, 2025

No description provided.

@Thodd Thodd force-pushed the chore/modules/cleanup-serializer-coding branch from 19103b3 to 31fd428 Compare August 15, 2025 14:01
type = typeInfo?.references?.[0]?.name || type;

if (type === "ValueState" && typeInfo.references?.[0].package === "@ui5/webcomponents-base") {
if (typeInfo.references?.[0].package === "@ui5/webcomponents-base") {
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@H4ze: This solves the issue of misinterpreting the UI5Element.
There is only one relevant change in the test fixture: ShellBar.json now has the correct type for the content aggregation

@Thodd Thodd requested review from H4ze and petermuessig and removed request for petermuessig August 19, 2025 07:38
@Thodd Thodd changed the title chore(ui5-tooling-modules): cleanup coding, better readable logging fix(ui5-tooling-modules): references to webc-base/UI5Element now point to core.WebComponent Aug 19, 2025
@Thodd Thodd requested a review from petermuessig August 19, 2025 09:09
Copy link
Member

@petermuessig petermuessig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@petermuessig petermuessig merged commit 5e67119 into main Aug 19, 2025
5 checks passed
@petermuessig petermuessig deleted the chore/modules/cleanup-serializer-coding branch August 19, 2025 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants