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

Tree and Property Widget: Cleanup internal APIs #885

Merged
merged 14 commits into from
Jun 10, 2024

Conversation

jasdom
Copy link
Contributor

@jasdom jasdom commented Jun 5, 2024

Closes #811.

@jasdom jasdom marked this pull request as ready for review June 5, 2024 13:33
@jasdom jasdom requested review from a team as code owners June 5, 2024 13:33
scripts/checkInternal.js Show resolved Hide resolved
@jasdom
Copy link
Contributor Author

jasdom commented Jun 6, 2024

Seems like React does not support passing props with symbols and the only workaround would be to use context facebook/react#7552. Due to this, reverted the change.

@jasdom jasdom enabled auto-merge (squash) June 7, 2024 14:17
@jasdom jasdom merged commit e74f9ca into master Jun 10, 2024
10 checks passed
@jasdom jasdom deleted the tree-and-property-widget-api-cleanup branch June 10, 2024 13: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.

Tree & Property widgets: Clean up internal APIs exposed through barrel file
4 participants