Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[UX] Update and Backup text improvements #6805

Open
jenlampton opened this issue Jan 3, 2025 · 1 comment · May be fixed by backdrop/backdrop#4987
Open

[UX] Update and Backup text improvements #6805

jenlampton opened this issue Jan 3, 2025 · 1 comment · May be fixed by backdrop/backdrop#4987

Comments

@jenlampton
Copy link
Member

jenlampton commented Jan 3, 2025

  1. When running updates, the first step of the process contains the paragraph For more detailed information, see the [Upgrading Backdrop CMS](https://backdropcms.org/upgrade) page. If you are unsure of what these terms mean, contact your hosting provider.

I would recommend removing the second sentence -- what is a hosting provider going to do if you don't understand these terms?

  1. The title of the backup page is Pre-update backup , both of these words are both verbs and nouns, so this is confusing in english and also may not translate well.

Could we change this to a complete sentence?

The title on both the previous and next step is Backdrop site update so maybe this page could be Backdrop backup process

something like "Create a backup"?

  1. This paragraph: If skipping the backup process, please ensure you create a backup through a different mechanism, such as through your hosting provider.
  • I am not skipping the "backup process" if I have done it elsewhere, just skipping it here.
  • If we remove 'ensure you` the intent is the same and the language is more simple.
  • I would not call a hosting provider a mechanism.
  • Hosting provider also assumes the update is being run remotely, but I run a lot more updates locally than remotely. (Does this matter?)

How about something like...
If skipping the backup process here, please create a backup elsewhere, such as through your hosting provider.

  1. This paragraph: Backups are not available because the variable $settings['backup_directory'] has not been set in settings.php.

I nearly missed this because it's below a bunch of other text, and formatted the same way as everything above. I would prefer to see this formatted as warning, or at least an info message, and maybe also displayed at the top of the page.

  1. this paragraph: The backup process may take several minutes, depending on the size of your database.

The comma is not necessary

Steps To Reproduce

To reproduce the behavior:

  1. Go to 'core/update.php'

Additional information

Add any other information that could help, such as:

  • Backdrop CMS version:
  • Web server and its version:
  • PHP version:
  • Database sever (MySQL or MariaDB?) and its version:
  • Operating System and its version:
  • Browser(s) and their versions:
@jenlampton jenlampton added this to the 1.30.0 milestone Jan 3, 2025
@jenlampton jenlampton self-assigned this Jan 8, 2025
@jenlampton
Copy link
Member Author

I've made the changes, I'm not sure this a good way to get the info messaging styling, but it seems to work 🤷‍♀️

Screenshot 2025-01-09 at 2 30 58 PM Screenshot 2025-01-09 at 2 31 55 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment