Problem/Motivation
Move forward with feedback received from @rkoller, please find original comment below:
"I have applied the latest patch and have taken a look at the micro copy in the context of admin/reports/updates/automatic-update and admin/automatic-update-ready/. I like and agree that you have added texts on both pages" I would have a few additional points and thoughts to add.
1. I am not sure if starting off by providing a gently and friendly reminder saying, it is a good idea to back up your database, and then in the next step state in bold letters, that it is strongly recommended to back up the database and site, isn't a bit of mixed messaging. I would rather go with an explicit and clear statement on the automatic-update page that it is highly recommended to back up the database and site and then phrase the ready to update text more as the friendly reminder by stating that the downloaded assets will be applied next and that this step cannot be undone, the reason why an update is a good idea. Just provide clear and actionable steps what is recommended in the first place. Currently it sounds more like an update might be a good idea but isn't necessary and then you read on the ready to update page it is recommended after all. So a more explicit variant for the automatic-update page could be:
It is strongly recommended to back up your database and site before you begin.
(*maybe that sentence in bold on the automatic-update page?)
2. On the automatic-update page you have "back up your database" on the ready to update page you have "back up your database and site". It is advisable to be consistent with the actionable recommendation imho. Currently if someone is following the recommendation on the first page and just backs up the database could be puzzled when retrying and then reading on the ready to update page that database AND site should be updated now. Might be even considered annoying for some people. I would suggest to go with the link text "backup your database and site" on both pages.
3. One question that is probably still valid from #4. If an update is triggered manually there is the recommendation to back up the database and site while when the update is triggered and run automatically via cron there is no recommendation. Manually you get the strong recommendation to back up in the micro copy while when run automatically there is no notification for that? That "might" cause also uncertainty once someone triggers a manual update?
4. On the ready to update page you have the first line "Drupal core will be updated to 9.3.15" in a regular weight while the next line is bold. The sole focus is on the bold typeface (at least for me the "Drupal core will be updated part..." was more or less "invisible" or at least unnoticed that way) . I would perhaps make the whole block in a regular typeface and phrase the back up your database part a little bit different. Not that strongly but explaining the reasoning a little bit. I've added the word step to "this cannot be undone. At the end i would provide a brief note about what happens when you hit the cancel update button, stating that the downloaded assets get removed (aka they have to be re-downloaded if the user triggers the update process again):
Drupal core will be updated to 9.3.15. This step cannot be undone. The reason why to back up your database and site before the update was recommended in the first place. If you cancel the update process the downloaded assets get removed.
5. I could offer to put the issue on fridays ux meeting agenda (there is no entry yet - https://www.drupal.org/project/drupal/issues/3284204). There would be a few more pair of eyes to take a look at the issue and the word smithing part in particular. But i don't know how time sensitive the issue is since it is a stable blocker. But i think it would be a good choice.
And if i remember correctly Aaron also suggested a few weeks back in one automatic update issue to bring the automatic updates module in general to one of the UX meetings so that we could take a look as a group."
Comment | File | Size | Author |
---|---|---|---|
#20 | Screenshot 2023-01-05 at 7.08.09 PM.png | 297.82 KB | yash.rode |
Issue fork automatic_updates-3284936
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
Theresa.GrannumComment #3
Theresa.GrannumComment #5
Theresa.GrannumPer feedback from @rkoller updated messages on UpdateForm and UpdateReady. Requesting further input from @tedbow before merge.
Comment #6
tedbowComment #7
Theresa.GrannumComment #8
phenaproximaHonestly, I have no objections to this, except for the lack of test coverage. Tagging for that, but otherwise I'm happy with this wording.
Comment #9
phenaproximaAddressed @tedbow's point in the merge request and added tests. So, assigning to him for final review.
Comment #10
rkollerI like the update page better with removing the
it is strongly recommended
sentence and moving it back to the next step. but i wonder would it make sense to makeIt is strongly recommended to back up your database and site code before you begin.
a warning message? that way it would visually highlighted? at the moment the message comes after the primary button. basically the user should made up his or her mind before clicking the button. i wonder also from a screen reader perspective what if a screen reader user processed everything before the button and then “might” miss the message after the primary key because pressing the continue button?and the sentence on the ready to update page i would split up into two.
This step cannot be undone!
as a separate sentence with an exclamation mark....before continuing,..
wouldn’t be really necessary it is already implied in the first sentence stating that the next step cannot be undone.and i guess
so it is a good idea
sounds a bit inconsistent. i know back then it wasthe good idea
sentence on the first page(dont know exact wording anymore) andthe highly recommended
sentence on the ready to update page. but withthis cannot be undone
in the sentence before andit is strongly recommended
on the update page before, ending withit is a good idea
sounds like it isnt that important and necessary after all. so perhaps something like the following would be more consistent concise overall:This step cannot be undone! It is still strongly recommended to back up your database and site code in case you haven't already done so.
Comment #11
rkoller...and setting the status back to needs work after asking @phenaproxima on Slack
Comment #12
wim leersThis is still relevant and actionable.
Let's address @rkoller's feedback from #10.
Comment #13
wim leersComment #14
tedbowI think this could still be done after MVP(though would like to get it done before)
Comment #15
yash.rode CreditAttribution: yash.rode at Acquia commentedComment #16
yash.rode CreditAttribution: yash.rode at Acquia commentedComment #17
wim leersOne question on the MR for you, @yash.rode 😊
Comment #18
yash.rode CreditAttribution: yash.rode at Acquia commentedComment #19
wim leersCan you please post a short screencast showing that without the
messenger->deleteByType()
call, two messages appear? 🙏 I don't see how that's possible for the reason mentioned on the MR 🫣Comment #20
yash.rode CreditAttribution: yash.rode at Acquia commentedComment #21
yash.rode CreditAttribution: yash.rode at Acquia commentedComment #22
wim leersWell … those "readiness checks" messages do seem pretty important. And they seem intentional. I don't see why we want to delete them? Also, are you sure that's what's happening inside the test?
Comment #23
yash.rode CreditAttribution: yash.rode at Acquia commentedEven if we have
we will still get the Readiness check messages, it will just delete
This step cannot be undone! It is still strongly recommended to back up your database and site code in case you haven't already done so.
message.Comment #24
wim leersHm … I'll need to manually test this then, I cannot reasonably RTBC this until I truly understand and confirm this. That'll have to wait until next week though!
Comment #25
tedbowComment #26
rajeevkumar CreditAttribution: rajeevkumar commented1. Clarify and Strengthen the Backup Recommendation:
Solution: Make the backup recommendation clearer and more consistent across both pages.
"It is strongly recommended to back up your database and site before beginning the update process." (Consider making this bold for emphasis.)
"The downloaded assets will be applied to your site in the next step. This action cannot be undone. Please ensure you have backed up your database and site before proceeding."
2. Ensure Backup Recommendation for Both Manual and Automatic Updates:
Solution: Add a backup recommendation for automatically triggered updates (via cron).