Skip to content

Fix external wrapper requiring id. #3183

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Feb 24, 2025
Merged

Fix external wrapper requiring id. #3183

merged 4 commits into from
Feb 24, 2025

Conversation

T4rk1n
Copy link
Contributor

@T4rk1n T4rk1n commented Feb 24, 2025

  • Fix the ExternalWrapper requiring an id.
  • Change ExternalWrapper props to component, componentPath.

@gvwilson gvwilson added P1 needed for current cycle fix fixes something broken labels Feb 24, 2025
@T4rk1n T4rk1n merged commit a9a5df5 into dash-3.0 Feb 24, 2025
1 of 3 checks passed
@T4rk1n T4rk1n deleted the fix-external-wrapper-id branch February 24, 2025 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gvwilson gvwilson gvwilson approved these changes

@alexcjohnson alexcjohnson Awaiting requested review from alexcjohnson

@ndrezn ndrezn Awaiting requested review from ndrezn

@emilykl emilykl Awaiting requested review from emilykl

Assignees

@T4rk1n T4rk1n

Labels
fix fixes something broken P1 needed for current cycle
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@T4rk1n @gvwilson