- Issue created by @xjm
- Status changed to Needs review
9 months ago 12:31am 6 June 2024 - πΊπΈUnited States xjm
I updated the draft based on the Drupal 11 beta, the release cycle changes, and the 10.3.0-beta1 and rc1 milestone notes. Also finally fleshed out the IS instructions.
- πΊπΈUnited States smustgrave
Thereβs currently 1 issues in RTBC. Itβs merged but release note needed some work and gave it a shot
- Status changed to Postponed
9 months ago 7:02pm 7 June 2024 - πΊπΈUnited States smustgrave
I'm assuming this is actually postponed
π Container compile crash when a service decorates a destructable service Needs work is in the list but is currently NW
π ImageStyleDownloadController routes do not limit schemes served Fixed is merged but release note needs a new review. - πΊπΈUnited States DamienMcKenna NH, USA
xjm β credited DamienMcKenna β .
- πΊπΈUnited States xjm
No, it is actively being worked on because the release comes out in less than a week. Thanks!
- Status changed to Needs review
9 months ago 3:36pm 13 June 2024 - πΊπΈUnited States smustgrave
Should the issue thatβs still in work be removed from 10.3 release notes
- πΊπΈUnited States neclimdul Houston, TX
Don't see it in the gdoc but π User logout is vulnerable to CSRF Fixed might be a disruptive change for sites. Had some hard coded /user/logout links in some templates which lead to a pretty ugly unthemed confirmation form.
- πΊπΈUnited States xjm
@smustgrave Which issue and which section? Stuff that is still underway is listed under "known issues" on purpose, but is there an issue outside those sections?
@neclimdul, thanks, I reopened π User logout is vulnerable to CSRF Fixed for a release note snippet.
- πΊπΈUnited States smustgrave
@xjm sorry I was referring to π Container compile crash when a service decorates a destructable service Needs work
- πΊπΈUnited States xjm
@smustgrave, Yep, that one is listed under the known issues on purpose, because it's a major bug that could cause problems for people updating to 10.3.
- πΊπΈUnited States xjm
Adding credit for @neclimdul; that's a good find.
- π¦πΊAustralia pameeela
Link to the comment π 10.3.0 release notes Active
Not much information to go on.
- πΊπΈUnited States nicxvan
I tagged two more issues:
π Pager h4 causes accessibility flag on many pages Fixed
π Remove country setting from the installer FixedRe reading this issue summary they probably belong in the highlights rather than release notes.
Let me know where they go and I'll retag them.
- πΊπΈUnited States nicxvan
Also π Stable9 accessibility update: Pager h4 causes accessibility flag on many pages RTBC was already tagged, but I don't see it in the release notes.
- πΊπΈUnited States xjm
@nicxvan, correct, only changes that are disruptive should go in the release notes. The release notes should describe the disruption the site owners or module developers need to address and how.
If it's just an improvement or feature, it should be tagged for the highlights instead.
I don't have time at the moment to look over these three issues, but if they are disruptive, then the release notes should be updated to describe what the disruption is, who is affected, and what they need to do to fix it, with a link to the change record. For more information and examples, see the release note snippet documentation β .
Leaving NR to check back on them.
- πΊπΈUnited States nicxvan
In that case the pager ones are likely highlights since they are accessibility improvements (They were split up since one of them was for stable9)
π Pager h4 causes accessibility flag on many pages Fixed
π Stable9 accessibility update: Pager h4 causes accessibility flag on many pages RTBC
I've retagged those two already.The country default in the installer is likely disruptive if someone is expecting that default to be set. It was not used in core and low usage in contrib, but likely qualifies for release notes:
π Remove country setting from the installer Fixed - πΊπΈUnited States smustgrave
Will say about the pager ones though. Even though they aren't disruptive any module that ships with a view config file will need to be updated for future installs right?
- π¦πΊAustralia pameeela
Edit: going to move my comment to the specific issue.
- πΊπΈUnited States nicxvan
After reading the comment from @pameeela I don't think the country setting needs a release note.
@smustgrave that is a good point, I think since we added config new config we didn't hit the circular issue @dww hit here: https://www.drupal.org/project/drupal/issues/2640994#comment-15597379 π Label token replacement for views entity reference arguments not working Needs work so these probably can stay in highlights unless someone disagrees.
I think my confusion stemmed from this comment: https://www.drupal.org/project/drupal/issues/3333418#comment-14886507 π Stable9 accessibility update: Pager h4 causes accessibility flag on many pages RTBC
and since that issue was a follow up I assumed the parent needed release notes too, but based on the docs I think they fit better under highlights. - Status changed to RTBC
9 months ago 12:40am 22 June 2024 - πΊπΈUnited States smustgrave
With 10.3.0 out should this be marked fixed?
- Status changed to Fixed
9 months ago 6:32am 22 June 2024 - π¬π§United Kingdom catch
Yep we can close this now (still possible to add issues to the release notes now the release it's out if something comes up).
- πΊπΈUnited States xjm
Automatically closed - issue fixed for 2 weeks with no activity.
- π³π±Netherlands idebr
It seems that many of the issues tagged with "10.3.0 release highlights" didn't make it to the release notes: https://www.drupal.org/project/issues/search?issue_tags=10.3.0%20release... β
Several issues I have ran into in customer projects weren't mentioned, for example:
π Redesign the menu link add form to be less overwhelming Fixed
π Move system/base component CSS to respective libraries where they exist Fixed
π Add taxonomy term revision UI RTBCCan these highlights be merged into the release notes?