-
Notifications
You must be signed in to change notification settings - Fork 16.7k
Add Refresh Token logic to auth manager docs#54196
Conversation
follow up docs
relates: #51657
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.
5706a31 to
2ade6a3
Compare
2ade6a3 to
9cf8339
Compare
pierrejeambrun
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions. |
|
Not stale, will update after the solution merged |
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions. |
|
Okay it is already time. I will update the doc in a day or two. Finishing some CI work to lose dependency on v3 test branches on integration tests |
|
I removed my request for change. Things have been discussed, implemented and stabilized, since then, we're ready for a doc update :) |
6085e4f to
58b6ef5
Compare
1e8f310 to
adfd095
Compare
adfd095 to
d3ac537
Compare
|
Docs and spellcheck are currently passing the error. Merging. Thanks for the reviews! The image /mnt/ci-image-save-v3-linux_amd64-3.10.tar does not exist.
|
d3ac537 to
912c5c9
Compare