Activity

1. kumaraditya303 commented on Oct 28, 2022

   

   kumaraditya303

   on Oct 28, 2022

   More actions

   cc @JulienPalard @CAM-Gerlach
2. JulienPalard commented on Oct 28, 2022

   

   JulienPalard

   on Oct 28, 2022

   Member

   More actions

   Thanks for opening this issue, it's interesting!

   It also happen in any other change of whatsnew, but when "decreasing" the version it lands you on the homepage as the target file does not exist.

   The page switch is handled client-side via this function:

   => https://github.com/python/docsbuild-scripts/blob/main/templates/switchers.js#L81

   so can you please reopen the issue (and or a PR) on the docsbuild-script repo?
3. CAM-Gerlach commented on Oct 28, 2022

   

   CAM-Gerlach

   on Oct 28, 2022

   Member

   More actions

   FYI @JulienPalard , you can simply transfer this issue directly to that repo with the (not easy to spot) Transfer Issue button at the bottom of the right sidebar, without having to recreate a whole new one (I can't as a triager, until the forthcoming permissions workaround is implemented):

   

   For me, this is the expected (if not necessarily ideal) behavior, not a "bad redirection" per say—each What's New is its own separate page, and there is no one "current version" What's New page, so naturally switching between different branches gets that branch's version of that What's New page,

   What this issue is really requesting is not a bug fix, but rather an enhancement to the behavior of the version switcher, to add special-case logic to more "intelligently" handle switching between "What's New" pages of different Python versions in a way that is more useful to most users. Of course, its more complicated than the issue initially implies, as there are a number of cases to consider and its not completely obvious what the ideal logic should be in every case, but if @JulienPalard thinks its worth it, then it would be an improvement. cc @hugovk

   If so, I propose the following logic, in psuedocode:

   # If on the What's New for the current Python version,
   # Navigate to the selected Python version's What's New page
   if current_version == current_whatsnew_version:
       new_whatsnew_version = selected_version
   # If the selected version is too early to have the current What's New version
   # fall back to the latest What's New version present (i.e. that version's)
   elif selected_version > whatsnew_version:
       new_whatsnew_version = selected_version
   # Otherwise, just keep the same What's New version, as now
   else:
       new_whatsnew_version = whatsnew_version
   

   One possible design for the implementation:

   • Add a function get_whatsnew_version() that takes a url, and returns the capture group in /whatsnew/(\d\d?\.\d\d?), or else null (or some falsely value)
   • Add a function switch_whatsnew_version() that takes the current, selected and what's new versions, .split('.')s them into arrays, applies the above logic (since JS array comparisons appear to work like Python tuple comparisons) and then returns the new what's new version re-knit into a string.
   • Add a function set_whatsnew_version() that takes a url and the new whatsnew_version and returns the the URL with /whatsnew/\d\d?\.\d\d? replaced with the new version.
   • Either stick all those into one function and call it from on_version_switch, or call get_whatsnew_version there and then call the other two in sequence to replace url if the result of the first is truthy
4. hugovk commented on Oct 28, 2022

   

   hugovk

   on Oct 28, 2022

   Member

   More actions

   Yeah, I've experienced this and found it a little surprising, albeit logical when you think about it for a moment.

   I think the common case would to expect a "sticky" What's New, as suggested.
5. JulienPalard

   transferred this issue frompython/cpythonon Oct 28, 2022
6. JulienPalard commented on Oct 28, 2022

   

   JulienPalard

   on Oct 28, 2022

   Member

   More actions

   FYI @JulienPalard , you can simply transfer this issue directly to that repo

   Wow thanks!

   🚀React with 🚀1Reacted by C.A.M. Gerlach
7. JulienPalard commented on Oct 28, 2022

   

   JulienPalard

   on Oct 28, 2022

   Member

   More actions

   I think we all agree that something better can be implemented.

   I'm not sure we need to complicate things more than just navigating to f"{selected_version}/whatsnew/{selected_version}.html", after all:

   • It's simple and predictible.
   • The page always exists.
   • If the reader choose a version from the picker while reading a whatsnew, it's highly probable that he wanted to see the choosen version (am I wrong?).
8. CAM-Gerlach commented on Oct 29, 2022

   

   CAM-Gerlach

   on Oct 29, 2022

   Member

   More actions

   I certainly understand the desire to not complicate the logic, which was my one concern with this proposal to begin with.

   That's essentially what the above proposed implementation boils down to (in perhaps a dozen or so total lines of JS for the whole thing), except that if the reader is browsing a What's New that isn't the current version's, and exists on the target, the What's New version isn't touched, as it wouldn't make sense to arbitrarily change it (given the reader deliberately navigated to that older version, which I would view as a regression).
9. ThisisYoYoDev commented on Oct 29, 2022

   

   ThisisYoYoDev

   on Oct 29, 2022

   Author

   More actions

   I completely agree with @JulienPalard what I wanted here and what I think everyone wants is to be redirected to the version they want.

   When I arrived on the Whatsnew of 3.11 and I saw that the version 3.12 was available in alpha I said to myself I want to see the news about 3.12

   So I think the best thing would be to keep it simple like Julien suggested in his message :)
10. hugovk commented on Jan 9, 2023

    

    hugovk

    on Jan 9, 2023

    Member

    More actions

    See also python/cpython#100734.