viiccwen commented on PR #55277:
URL: https://github.com/apache/airflow/pull/55277#issuecomment-3263843196
> Key-value (implicit) If `conf` is not specified, any query parameter
that is not a reserved keyword will be collected into `conf`.
`/dags/{dag_id}/trigger/single?run_id=myrun&foo=bar&x=123` →
`conf={"foo":"bar","x":"123"}`
Didn't the PR description solve the problem? 🤔
After I read all the PRs you mentioned, I realized we need add `key=value`
to let user to fill the `conf` parameter. (in commit: [feat: supports conf
key=value key in method if not provided
explici…](https://github.com/apache/airflow/pull/55277/commits/26d8a7b796c7766066a3c6f85fc516c8c54128f7))
and I also updated the docs (in commit: [docs(params): add conf params usage
table between JSON and
`key=value`](https://github.com/apache/airflow/pull/55277/commits/969a418b708f4f5e38ed17c6dc9301dcb58f4a0e))
Now `conf` supports **2 types**: JSON & `key=value` (I think we could keep
JSON type for devs), you can check out the table on my PR description, and with
the video to represent it.
If I still misunderstood the meaning, sorry for my poor understanding... 🤡
--
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]
