Add link to Status Report from database updates completion page

Created on 22 February 2025, about 1 month ago

After database updates have been run from Drupal's admin UI, it would be swell if there was a link to the Status Report, please.

thanks,
-mike

✨ Feature request
Status

Active

Version

11.1 πŸ”₯

Component

database update system

Created by

πŸ‡ΊπŸ‡ΈUnited States ultimike Florida, USA

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @ultimike
  • First commit to issue fork.
  • Pipeline finished with Success
    about 1 month ago
    Total: 1779s
    #431688
  • πŸ‡ΊπŸ‡ΈUnited States nicxvan

    The code changes look good, since this is a front end change a screenshot is needed I think.

    I added the needs issue summary tag to get the standard issue template back, it's helpful to have all sections to review as issues progress.

    I did confirm that the administration pages links are tested elsewhere.

  • Since @ultimike asked for this...

  • πŸ‡ΊπŸ‡ΈUnited States ultimike Florida, USA

    @nicxvan, @cilefen,

    Wow - thanks so much for hopping on this so quickly!

    Some background on this request: each year at Florida DrupalCamp, during lightning talks, I share my list of "Drupal grievances" from the previous 12 months. The lack of the "Status Report" link was one of my original grievances a few years ago. At the time, I found a different core issue (which I have been unable to find again) that mentioned this, but its scope was bigger and never went anywhere.

    Fast-forward to a few days ago, and I decided that this would not only make a good item for my "grievance rewind" for this year's lightning talk, but also a good issue to tackle (and possibly mentor someone new on) for Florida Drupal Camp's contribution day. Imagine my surprise (and joy!) when I opened this issue a few minutes ago to see the MR!

    Hopefully, the narrow scope of this issue will allow this MR to be merged.

    Marking as RTBC.

    -mike

  • πŸ‡ΊπŸ‡ΈUnited States nicxvan

    I agree it's a good change, it's still needs work though because issues affecting the UI need before and after screenshots.

    I added the default template back in.

    If a section of the template is not applicable you can just leave it blank rather than deleting it, it makes it easier to see what is missing.

  • πŸ‡ΊπŸ‡ΈUnited States nicxvan
  • πŸ‡ΊπŸ‡ΈUnited States ultimike Florida, USA

    @nicxvan - just to clarify, are you saying that all that is currently needed is a before-and-after screenshot (and perhaps some better information in the issue template)?

    -mike

  • πŸ‡ΊπŸ‡ΈUnited States nicxvan

    Yep! Just a before and after screenshot.

    I already added the template back into the issue summary.

  • πŸ‡ΊπŸ‡ΈUnited States ultimike Florida, USA

    Screenshots added.

  • πŸ‡ΊπŸ‡ΈUnited States nicxvan

    Looks great! Not sure if this needs a change record or not.

  • πŸ‡¦πŸ‡ΊAustralia mstrelan

    I have reviewed the MR and the code looks good to add the link to the page, however I suggest opening a follow up issue to convert both links to use Url::access instead of User::hasPermission. Currently if the required permission changed to "access site reports", for example, the test would still pass even though the link would not work. Since we're already using this pattern I suggest we keep it for now and get this one in. Looks good to me.

    • quietone β†’ committed 6d097f18 on 11.x
      Issue #3508449 by cilefen, ultimike, nicxvan, mstrelan: Add link to...
  • πŸ‡¦πŸ‡ΊAustralia larowlan πŸ‡¦πŸ‡ΊπŸ.au GMT+10
  • πŸ‡³πŸ‡ΏNew Zealand quietone

    Nice little improvement!

    Thanks to @larowlan for manual testing. And to all the help at DrupalSouth 2025.

    Committed 6d097f1 and pushed to 11.x. Thanks!

  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024