Skip to content

fix: specify unit (MB) for the memory values in system info page#2308

Merged
nicomiguelino merged 1 commit intoScreenly:masterfrom
nicomiguelino:specify-unit-for-system-info-memory
May 31, 2025
Merged

fix: specify unit (MB) for the memory values in system info page#2308
nicomiguelino merged 1 commit intoScreenly:masterfrom
nicomiguelino:specify-unit-for-system-info-memory

Conversation

@nicomiguelino
Copy link
Contributor

Issues Fixed

The System Info page doesn't specify the unit (whether it's KB, MB, or GB) for the memory values.

Description

  • Update the System Info page so that the unit is specified (i.e., MB).

Checklist

  • I have performed a self-review of my own code.
  • New and existing unit tests pass locally and on CI with my changes.
  • I have done an end-to-end test for Raspberry Pi devices.
  • I have tested my changes for x86 devices.
  • I added a documentation for the changes I have made (when necessary).

Screenshots

Before

image

After

image

@sonarqubecloud
Copy link

@nicomiguelino nicomiguelino marked this pull request as ready for review May 31, 2025 05:45
@nicomiguelino nicomiguelino requested a review from a team as a code owner May 31, 2025 05:45
@nicomiguelino nicomiguelino merged commit ea39e55 into Screenly:master May 31, 2025
6 checks passed
@nicomiguelino nicomiguelino deleted the specify-unit-for-system-info-memory branch May 31, 2025 05:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant