Skip to content

Conversation

@Zeglow
Copy link

@Zeglow Zeglow commented Sep 28, 2025

Fix: Make Airbyte connection fields properly labeled in Airflow 3

Fixes the formatting issue in get_ui_field_behaviour method that was preventing proper field labeling for Airbyte connections in Airflow 3.

Problem:
In Airflow 3, the Airbyte provider connection form shows generic field names (login, password, schema) instead of user-friendly labels (Client ID, Client Secret, Token URL), making it confusing for users to set up Airbyte connections.

Root Cause:
The get_ui_field_behaviour method had incorrect formatting/indentation in the relabeling dictionary, causing the UI field relabeling functionality to fail silently.

Solution:

  • Fixed formatting and indentation in the relabeling dictionary
  • Ensures proper field labeling:
    • "host" → "Server URL"
    • "login" → "Client ID"
    • "password" → "Client Secret"
    • "schema" → "Token URL"

Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

- Add proper relabeling for Airbyte-specific field meanings
- Ensure 'login' displays as 'Client ID' and 'password' as 'Client Secret'
- Similar pattern to other provider fixes

Closes apache#55841
- Test that correct fields are hidden (extra, port)
- Test that proper relabeling is applied:
  - host -> Server URL
  - login -> Client ID
  - password -> Client Secret
  - schema -> Token URL
- Test that placeholders is empty
- Ensures the UI field behavior works correctly for Airbyte connections
@Zeglow
Copy link
Author

Zeglow commented Sep 28, 2025

Duplicate of #55869, closing this one.

@Zeglow Zeglow closed this Sep 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant