Skip to content

Conversation

@Pierre-Gilles
Copy link
Contributor

Pull Request check-list

To ensure your Pull Request can be accepted as fast as possible, make sure to review and check all of these items:

  • If your changes affects code, did your write the tests?
  • Are tests passing? (npm test on both front/server)
  • Is the linter passing? (npm run eslint on both front/server)
  • Did you run prettier? (npm run prettier on both front/server)
  • If you are adding a new features/services, did you run integration comparator? (npm run compare-translations on front)
  • Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to the community (forum) for testing before merging.
  • If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code)
  • If you are adding a new features/services which needs explanation, did you modify the user documentation? See the GitHub repo and the website.
  • Did you add fake requests data for the demo mode (front/src/config/demo.js) so that the demo website is working without a backend? (if needed) See https://demo.gladysassistant.com.

NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.

Description of change

Make dashboard & scene larger on large screens

@cloudflare-workers-and-pages
Copy link

Deploying gladys-plus with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1af90da
Status: ✅  Deploy successful!
Preview URL: https://8786f8c1.gladys-plus.pages.dev
Branch Preview URL: https://larger-container-scene-dashb.gladys-plus.pages.dev

View logs

@codecov
Copy link

codecov bot commented Dec 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.66%. Comparing base (6ecf1a6) to head (1af90da).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2381   +/-   ##
=======================================
  Coverage   98.66%   98.66%           
=======================================
  Files         914      914           
  Lines       15735    15735           
=======================================
  Hits        15525    15525           
  Misses        210      210           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@relativeci
Copy link

relativeci bot commented Dec 8, 2025

#3583 Bundle Size — 11.01MiB (~+0.01%).

1af90da(current) vs 6ecf1a6 master#3579(baseline)

Warning

Bundle contains 2 duplicate packages – View duplicate packages

Bundle metrics  Change 3 changes Regression 2 regressions
                 Current
#3583
     Baseline
#3579
Regression  Initial JS 6.09MiB(~+0.01%) 6.09MiB
Regression  Initial CSS 309.45KiB(+0.13%) 309.04KiB
Change  Cache Invalidation 67.15% 0%
No change  Chunks 51 51
No change  Assets 175 175
No change  Modules 1617 1617
No change  Duplicate Modules 21 21
No change  Duplicate Code 0.95% 0.95%
No change  Packages 136 136
No change  Duplicate Packages 2 2
Bundle size by type  Change 2 changes Regression 2 regressions
                 Current
#3583
     Baseline
#3579
Regression  JS 7.98MiB (~+0.01%) 7.98MiB
No change  IMG 2.58MiB 2.58MiB
Regression  CSS 327.3KiB (+0.12%) 326.89KiB
No change  Fonts 93.55KiB 93.55KiB
No change  Other 18.45KiB 18.45KiB
No change  HTML 13.58KiB 13.58KiB

Bundle analysis reportBranch larger-container-scene-dashboardProject dashboard


Generated by RelativeCIDocumentationReport issue

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants