Skip to content
This repository has been archived by the owner on Sep 16, 2024. It is now read-only.

fix: doesn't process read-only attribute in res def #1979

Merged
merged 1 commit into from
Jan 22, 2024

Conversation

thxCode
Copy link
Member

@thxCode thxCode commented Jan 20, 2024

Problem:

the generated UI schema doesn't process the read-only attribute.

Solution:

merge read-only between multiple templates' schemas.

Related Issue:

#1970

@thxCode thxCode changed the title fix: hasn't merged readonly in res def fix: doesn't process read-only attribute in res def Jan 20, 2024
@thxCode thxCode force-pushed the fix-1970 branch 2 times, most recently from 744b2d9 to c1c066d Compare January 20, 2024 09:51
Copy link
Contributor

@gitlawr gitlawr left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@shanewxy shanewxy left a comment

Choose a reason for hiding this comment

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

LGTM

@shanewxy shanewxy merged commit a1ad37f into seal-io:main Jan 22, 2024
2 checks passed
@thxCode thxCode deleted the fix-1970 branch January 22, 2024 07:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants