Skip to content

Add connection duplication action to UI - #75097

Closed
abhijeets25012-tech wants to merge 4 commits into
apache:mainfrom
abhijeets25012-tech:fix-74998-clean
Closed

abhijeets25012-tech wants to merge 4 commits into
apache:mainfrom
abhijeets25012-tech:fix-74998-clean

Conversation

@abhijeets25012-tech

Copy link
Copy Markdown
Contributor

Adds a Duplicate action to the Connections page in the Airflow UI. The action opens a prefilled connection form using the existing connection's details, while leaving the connection ID and password blank so the user can enter a new ID and credentials before creating the duplicate.

Adds the corresponding English translation for the Duplicate action.

@abhijeets25012-tech

Copy link
Copy Markdown
Contributor Author

Hi, I’ve implemented the connection duplication action in the Airflow UI. The Duplicate button opens a prefilled form, allowing users to create a new connection with a new connection ID. The password is left blank for security.

I’ve kept the changes limited to the Connections UI and its English translation. I’ll share the PR for review once the branch is finalized.

@yuseok89 yuseok89 left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I left a question about masked values. The UI static check also seems to be failing on formatting.

@abhijeets25012-tech

Copy link
Copy Markdown
Contributor Author

Hi maintainers,

I've addressed the changes and verified that all CI checks are passing. Could you please review the latest updates when you have a chance?

Thank you for your time and feedback!

@bbovenzi

Copy link
Copy Markdown
Contributor

Closing in favor of #75110

For any new UI, we do like to have screenshots.

@bbovenzi bbovenzi closed this Oct 19, 2026
@abhijeets25012-tech

Copy link
Copy Markdown
Contributor Author

Thanks for the update. I understand that this is being handled in #75110. I had completed the implementation and all relevant checks were passing, so I'd appreciate understanding whether there were any specific technical concerns with my approach, or whether the PR was closed solely to consolidate duplicate work. I'd be happy to contribute any missing pieces, including screenshots, to #75110.

@bbovenzi

Copy link
Copy Markdown
Contributor

Mainly duplicate work and we had to pick one. The one we merged had before/after screenshots and included an alert to let users know when they need to refill in any sensitive fields.

@abhijeets25012-tech

Copy link
Copy Markdown
Contributor Author

Thanks for clarifying, Brent. That makes sense. I understand the need to consolidate duplicate work, and I appreciate the feedback about before/after screenshots and the alert for sensitive fields. I'll make sure to include these UX details in future UI contributions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:translations area:UI Related to UI/UX. For Frontend Developers. translation:default

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants