hussein-awala opened a new pull request, #30784:
URL: https://github.com/apache/airflow/pull/30784

   related: #30778
   
   curretly for the operators sub class of `BaseOperator`, we provide the field 
value to super as `conn_id`  which is a template field, and the field is 
rendered in execution time and everything works fine, but in the 
`rendered-templates` view, we have the original field name with the jinja 
template before rendering.
   
   This PR add a new field for `BaseOperator` to specify the conn field name, 
and use it directly from the ancestor with needing to provide it to parent 
class, then we can add the new conn field name to `template_fields` to render 
it.
   
   Before:
   ![Screenshot from 2023-04-21 
02-49-25](https://user-images.githubusercontent.com/21311487/233516440-8b55d9f1-dcc2-4ec0-865f-e0964d0b94f4.png)
   
   After:
   ![Screenshot from 2023-04-21 
02-51-04](https://user-images.githubusercontent.com/21311487/233516459-fd5d6e14-7cc4-4ece-b1e0-70191c96caec.png)
   
   And they both works fine in the hook.
   
   P.S: I just update the not deprecated operators, and this change is b/c, so 
if the new field is not defined, and the `conn_id` is provider to parent class, 
it will work as before.
   


-- 
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]

Reply via email to