Skip to content

Fix Edit Connection dialog: add lazyMount to prevent JSON editor infinite loading#65969

Merged
bbovenzi merged 1 commit into
apache:mainfrom
shashbha14:fix/edit-connection-json-loading-65954
May 12, 2026
Merged

Fix Edit Connection dialog: add lazyMount to prevent JSON editor infinite loading#65969
bbovenzi merged 1 commit into
apache:mainfrom
shashbha14:fix/edit-connection-json-loading-65954

Conversation

@shashbha14
Copy link
Copy Markdown
Contributor

Closes #65954

'EditConnectionButton' was missing 'lazyMount' and 'unmountOnExit' on 'Dialog.Root', causing the Monaco JSON editor to mount hidden with zero dimensions and get stuck on "Loading...".
Added both props to match AddConnectionButton which works correctly.

I used Claude (claude.ai) as an AI assistant for parts of this implementation.

@boring-cyborg boring-cyborg Bot added the area:UI Related to UI/UX. For Frontend Developers. label Apr 27, 2026
@shashbha14 shashbha14 marked this pull request as ready for review April 28, 2026 15:37
@bbovenzi bbovenzi added this to the Airflow 3.2.2 milestone Apr 28, 2026
@bbovenzi bbovenzi added the backport-to-v3-2-test Mark PR with this label to backport to v3-2-test branch label Apr 28, 2026
@potiuk potiuk added the ready for maintainer review Set after triaging when all criteria pass. label May 5, 2026
@vatsrahul1001
Copy link
Copy Markdown
Contributor

LGTM, @bbovenzi can you review this as well?

@bbovenzi bbovenzi merged commit 2064a76 into apache:main May 12, 2026
154 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

Backport successfully created: v3-2-test

Note: As of Merging PRs targeted for Airflow 3.X
the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.

In matter of doubt please ask in #release-management Slack channel.

Status Branch Result
v3-2-test PR Link

github-actions Bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request May 12, 2026
… editor infinite loading (apache#65969)

(cherry picked from commit 2064a76)

Co-authored-by: Shashwati Bhattacharyaa <166847329+shashbha14@users.noreply.github.com>
Co-authored-by: Shashwati <shashwatibhattacaharya21.2@gmail.com>
aws-airflow-bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request May 12, 2026
… editor infinite loading (apache#65969)

(cherry picked from commit 2064a76)

Co-authored-by: Shashwati Bhattacharyaa <166847329+shashbha14@users.noreply.github.com>
Co-authored-by: Shashwati <shashwatibhattacaharya21.2@gmail.com>
bbovenzi pushed a commit that referenced this pull request May 12, 2026
… editor infinite loading (#65969) (#66828)

(cherry picked from commit 2064a76)

Co-authored-by: Shashwati Bhattacharyaa <166847329+shashbha14@users.noreply.github.com>
Co-authored-by: Shashwati <shashwatibhattacaharya21.2@gmail.com>
vatsrahul1001 pushed a commit that referenced this pull request May 20, 2026
… editor infinite loading (#65969) (#66828)

(cherry picked from commit 2064a76)

Co-authored-by: Shashwati Bhattacharyaa <166847329+shashbha14@users.noreply.github.com>
Co-authored-by: Shashwati <shashwatibhattacaharya21.2@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:UI Related to UI/UX. For Frontend Developers. backport-to-v3-2-test Mark PR with this label to backport to v3-2-test branch ready for maintainer review Set after triaging when all criteria pass.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Webserver edit connections Extra Fields JSON doesn't load

4 participants