-
Notifications
You must be signed in to change notification settings - Fork 16.3k
Only show latest minor versions in version drop-down in docs #57243
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
7972c7a to
09c654d
Compare
jscheffl
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.
I think this is a good solution! I like it.
gopidesupavan
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.
Yes indeed, definitely worth limiting :) LGTM
|
Let's wait for others / devlist comments -> this is quite a change. It might be a little confusing for users (I imagine) - and I am not entirely sure if 10 last versions is good cut-off ? |
I think when all the non-latest minor are off then we can get back to the full list in my view... At least every 6 months I have a reason to check for 1.10 docs :-D Maybe a "all other versions..." can be added in the dropdown leading to a version browser page that lists all in a scrollable page? |
That's "advanced" UI work - that no-one pivcked up yet :) |
09c654d to
e045634
Compare
This does not remove the actual versions from S3 only limits the drop-down. All versions are available still to see and they are indexed, also links from PyPI continue to work, you can also manually update version in the URL.
e045634 to
a64ed77
Compare
|
The setup for 20 versions seems like really good for now :) |
…ocs (#57243) This does not remove the actual versions from S3 only limits the drop-down. All versions are available still to see and they are indexed, also links from PyPI continue to work, you can also manually update version in the URL. (cherry picked from commit a0b4d29) Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
…ocs (#57243) (#57262) This does not remove the actual versions from S3 only limits the drop-down. All versions are available still to see and they are indexed, also links from PyPI continue to work, you can also manually update version in the URL. (cherry picked from commit a0b4d29) Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
Cool! |
This does not remove the actual versions from S3 only limits the drop-down. All versions are available still to see and they are indexed, also links from PyPI continue to work, you can also manually update version in the URL.
^ 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.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.