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."

CommentFileSizeAuthor
#20 Screenshot 2023-01-05 at 7.08.09 PM.png297.82 KByash.rode
Command icon 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

Theresa.Grannum created an issue. See original summary.

Theresa.Grannum’s picture

Issue tags: -https://www.drupal.org/project/automatic_updates/issues/3239759
Related issues: +#3239759: Remind users to back up their database before starting an update
Theresa.Grannum’s picture

Title: Improve messaging to backup your database » Improve messaging to users to backup database

Theresa.Grannum’s picture

Per feedback from @rkoller updated messages on UpdateForm and UpdateReady. Requesting further input from @tedbow before merge.

tedbow’s picture

Status: Active » Needs work
Theresa.Grannum’s picture

Status: Needs work » Needs review
phenaproxima’s picture

Status: Needs review » Needs work
Issue tags: +Needs tests

Honestly, I have no objections to this, except for the lack of test coverage. Tagging for that, but otherwise I'm happy with this wording.

phenaproxima’s picture

Assigned: Theresa.Grannum » tedbow
Status: Needs work » Needs review
Issue tags: -Needs tests

Addressed @tedbow's point in the merge request and added tests. So, assigning to him for final review.

rkoller’s picture

I like the update page better with removing theit is strongly recommended sentence and moving it back to the next step. but i wonder would it make sense to make It 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 was the good idea sentence on the first page(dont know exact wording anymore) and the highly recommended sentence on the ready to update page. but with this cannot be undone in the sentence before and it is strongly recommended on the update page before, ending with it is a good ideasounds 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.

rkoller’s picture

Status: Needs review » Needs work

...and setting the status back to needs work after asking @phenaproxima on Slack

wim leers’s picture

Assigned: tedbow » Unassigned
Issue tags: +String change, +Usability

This is still relevant and actionable.

Let's address @rkoller's feedback from #10.

wim leers’s picture

Issue tags: +sprint
tedbow’s picture

Issue tags: +core-post-mvp

I think this could still be done after MVP(though would like to get it done before)

yash.rode’s picture

Assigned: Unassigned » yash.rode
yash.rode’s picture

Assigned: yash.rode » wim leers
Status: Needs work » Needs review
wim leers’s picture

Assigned: wim leers » yash.rode

One question on the MR for you, @yash.rode 😊

yash.rode’s picture

Assigned: yash.rode » wim leers
wim leers’s picture

Assigned: wim leers » yash.rode
Status: Needs review » Needs work

Can 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 🫣

yash.rode’s picture

StatusFileSize
new297.82 KB

message on update ready page

yash.rode’s picture

Assigned: yash.rode » wim leers
Status: Needs work » Needs review
wim leers’s picture

Assigned: wim leers » yash.rode
Status: Needs review » Needs work

Well … 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?

yash.rode’s picture

Assigned: yash.rode » wim leers
Status: Needs work » Needs review

Even if we have

// Delete warnings added on updater form.
    $this->messenger()->deleteByType(MessengerInterface::TYPE_WARNING);

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.

wim leers’s picture

Hm … 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!

tedbow’s picture

Version: 8.x-2.x-dev » 3.0.x-dev
rajeevkumar’s picture

1. Clarify and Strengthen the Backup Recommendation:
Solution: Make the backup recommendation clearer and more consistent across both pages.

  • Start with a clear and firm recommendation. For example, update the text to:
    "It is strongly recommended to back up your database and site before beginning the update process." (Consider making this bold for emphasis.)
  • Phrase this as a friendly reminder, mentioning that the update is imminent. Example:
    "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).

  • Add a reminder to back up your database and site before automatic updates are triggered. This can be phrased as a part of the notification when the update is initiated automatically:
  • "Before proceeding with the automatic update, ensure you have backed up your database and site."