amoghrajesh commented on PR #60410:
URL: https://github.com/apache/airflow/pull/60410#issuecomment-3817454703
@potiuk @jason810496 @jscheffl I made some changes to the PR and would like
another set of eyes:
1. I removed the interim container class (mock classes) and am directly
converting the yaml path response to be that API expects
2. Added a tool for helping with migration:
`scripts/tools/generate_yaml_format_for_hooks.py` and doc for it:
`contributing-docs/23_provider_hook_migration_to_yaml.rst`, one example of how
it works:
```shell
(apache-airflow) ➜ airflow git:(use-yaml-to-load-ui-conn-form) ✗ python
scripts/tools/generate_yaml_format_for_hooks.py --provider docker
Processing: airflow.providers.docker.hooks.docker.DockerHook
# airflow.providers.docker.hooks.docker.DockerHook (connection_type: docker)
ui-field-behaviour:
hidden-fields:
- schema
relabeling:
host: Registry URL
login: Username
placeholders:
extra: '{"reauth": false, "email": "[email protected]"}'
```
--
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]
