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

Follow-up: display new install_version property also on status and debug page #6798

Open
indigoxela opened this issue Dec 31, 2024 · 3 comments · May be fixed by backdrop/backdrop#4964
Open

Comments

@indigoxela
Copy link
Member

indigoxela commented Dec 31, 2024

Description of the task

As part of (and basis for) supplemental stylesheets in issue #4782 there's now a new config item install_version, which has already been added to our telemetry.

Besides that, this information would also be beneficial on the status page (admin/reports/status) and on the debug information page (admin/reports/debug).

Actual behavior

On admin/reports/telemetry the information's displayed but neither on admin/reports/status nor on admin/reports/debug.

Expected behavior

This info would be useful on the other two pages, too. For sure on the debug info page.

Additional information

Add any other information that could help, such as:

  • Backdrop CMS version: 1.30.x (not released yet)
@stpaultim
Copy link
Member

image

I like this idea.

One thought. Should we include a more option with text along these lines.

Starting with version 1.30.0, Backdrop uses the installation version to add classes to the body of Basis which makes it possible to improve Basis without effecting older sites.

This is pretty obscure and not likely to be seen by many people, but it might be helpful to a few.

I don't feel strongly about this suggestion.

@indigoxela
Copy link
Member Author

One thought. Should we include a more option with text along these lines.

We could add more info there, however, as your text already shows: that's only relevant if Basis is the default theme. That may be the case for some sites, but most of the time rather not. And for these sites, that info would rather be confusing than helpful.

Besides that, there's a discussion to add a UI for that Basis setting - which will provide more helpful background info than a status page "more" section".

So - we can do it - if there's consensus that this makes sense.

@avpaderno
Copy link
Member

avpaderno commented Jan 1, 2025

The Backdrop version installed when the site was created can be useful in other cases. For example, it could be helpful to know if a site was created with Backdrop CMS 1.34, or it was updated to Backdrop CMS 1.34 and, in the latter case, if it was updated from Backdrop CMS 1.31 or from Backdrop CMS 1.33.
Describing Backdrop CMS initial version as a value that can explain why a theme is rendered differently in two different sites would be too restrictive.

I am not sure what additional text could be added. I would eventually make clear what Backdrop CMS initial version is, just to avoid the eventual confusion Backdrop CMS and Backdrop CMS initial version could create. (The confusion would stop when Backdrop CMS is updated to a new version, though.)

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

Successfully merging a pull request may close this issue.

3 participants