Taragolis commented on code in PR #33822:
URL: https://github.com/apache/airflow/pull/33822#discussion_r1307714248
##########
airflow/providers/microsoft/azure/hooks/container_volume.py:
##########
@@ -54,14 +56,22 @@ def _get_field(self, extras, name):
@staticmethod
def get_connection_form_widgets() -> dict[str, Any]:
"""Returns connection widgets to add to connection form."""
- from flask_appbuilder.fieldwidgets import BS3PasswordFieldWidget
+ from flask_appbuilder.fieldwidgets import BS3PasswordFieldWidget,
BS3TextFieldWidget
from flask_babel import lazy_gettext
- from wtforms import PasswordField
+ from wtforms import PasswordField, StringField
return {
"connection_string": PasswordField(
lazy_gettext("Blob Storage Connection String (optional)"),
widget=BS3PasswordFieldWidget()
),
+ "subscription_id": StringField(
+ lazy_gettext("Subscription ID (optional)"),
+ widget=BS3TextFieldWidget(),
+ ),
+ "resource_group": StringField(
+ lazy_gettext("Resource group name (optional)"),
+ widget=BS3TextFieldWidget(),
+ ),
Review Comment:
Would be nice to have information about this extra fields into the
documentation
https://github.com/apache/airflow/blob/b82ce61285f3f4f0c7eccb2c3effaef53c9fb84e/docs/apache-airflow-providers-microsoft-azure/connections/azure_container_volume.rst?plain=1#L60-L64
--
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]
