WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@D8H
Copy link
Collaborator

@D8H D8H commented Nov 28, 2025

No description provided.

@D8H D8H requested a review from 4ian as a code owner November 28, 2025 15:23
@D8H D8H force-pushed the custom-object-advanced-property branch from b41dc27 to 831404c Compare December 3, 2025 09:27
const areAdvancedPropertiesExpandedByDefault = React.useMemo(
() => areAdvancedPropertiesModified(behavior),
() => {
const behaviorMetadata = gd.MetadataProvider.getBehaviorMetadata(
Copy link
Owner

Choose a reason for hiding this comment

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

Is there a risk these are "bad metadata"? Should we protect against this?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I guess that with the other changes done, it should be fine because it will get '' for any properties.

Copy link
Owner

Choose a reason for hiding this comment

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

Yes that should be fine!

@D8H D8H merged commit f01eb6d into master Dec 3, 2025
5 of 6 checks passed
@D8H D8H deleted the custom-object-advanced-property branch December 3, 2025 12:27
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