You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/01/19 22:44:35 UTC
[GitHub] [airflow] bbovenzi opened a new pull request #13778: plain text description for conn type
bbovenzi opened a new pull request #13778:
URL: https://github.com/apache/airflow/pull/13778
Add a plaintext description to add/edit conn_type to make sure people remember to install necessary provider packages.
Closes https://github.com/astronomer/issues/issues/2520
<img width="814" alt="Screen Shot 2021-01-19 at 12 44 38 PM" src="https://user-images.githubusercontent.com/4600967/105103073-8d1b9280-5a75-11eb-947c-acc2115ac696.png">
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] mik-laj removed a comment on pull request #13778: plain text description for conn type
Posted by GitBox <gi...@apache.org>.
mik-laj removed a comment on pull request #13778:
URL: https://github.com/apache/airflow/pull/13778#issuecomment-763796233
Can you add this field to REST API? This will allow us to keep the feature parity between Web UI and API.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] mik-laj commented on pull request #13778: plain text description for conn type
Posted by GitBox <gi...@apache.org>.
mik-laj commented on pull request #13778:
URL: https://github.com/apache/airflow/pull/13778#issuecomment-763796233
Can you add this field to REST API?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] kaxil merged pull request #13778: plain text description for conn type
Posted by GitBox <gi...@apache.org>.
kaxil merged pull request #13778:
URL: https://github.com/apache/airflow/pull/13778
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] mik-laj edited a comment on pull request #13778: plain text description for conn type
Posted by GitBox <gi...@apache.org>.
mik-laj edited a comment on pull request #13778:
URL: https://github.com/apache/airflow/pull/13778#issuecomment-763796233
Can you add this field to REST API? This will allow us to keep the feature parity between Web UI and API.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] github-actions[bot] commented on pull request #13778: plain text description for conn type
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #13778:
URL: https://github.com/apache/airflow/pull/13778#issuecomment-763192295
The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not modify the core of Airflow. If the committers decide that the full tests matrix is needed, they will add the label 'full tests needed'. Then you should rebase to the latest master or amend the last commit of the PR, and push it with --force-with-lease.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on pull request #13778: plain text description for conn type
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #13778:
URL: https://github.com/apache/airflow/pull/13778#issuecomment-763682605
Nice! Good idea!
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org