shahar1 commented on code in PR #44598:
URL: https://github.com/apache/airflow/pull/44598#discussion_r1873758508
##########
providers/src/airflow/providers/hashicorp/hooks/vault.py:
##########
@@ -385,7 +368,6 @@ def get_connection_form_widgets(cls) -> dict[str, Any]:
description="Must be 1 or 2.",
default=DEFAULT_KV_ENGINE_VERSION,
),
- "role_id": StringField(lazy_gettext("Role ID (deprecated)"),
widget=BS3TextFieldWidget()),
Review Comment:
What are you confused about? Whether we should remove this line?
If you referred to that, I think that it's ok to remove it as it's a part of
the deprecated features. If you meant something else, or wrote this comment
here by mistake, please let me know.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
