Skip to content

Make screens immediately update before polling period elapses - #3862

Merged
ryanmelt merged 1 commit into
mainfrom
immediate_screen_update
Sep 15, 2026
Merged

ryanmelt merged 1 commit into
mainfrom
immediate_screen_update

Conversation

@ryanmelt

Copy link
Copy Markdown
Member

What changed

See title

Why it changed

A screen with a 5 second period just showed nulls until after 5 seconds.

Testing strategy

Manually tested

@codecov

codecov Bot commented Sep 13, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 79.63%. Comparing base (aa0c194) to head (e569828).

Files with missing lines Patch % Lines
.../openc3-vue-common/src/components/Openc3Screen.vue 66.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3862      +/-   ##
==========================================
+ Coverage   79.58%   79.63%   +0.05%     
==========================================
  Files         901      901              
  Lines       68067    68070       +3     
  Branches     2633     2635       +2     
==========================================
+ Hits        54171    54208      +37     
+ Misses      13222    13191      -31     
+ Partials      674      671       -3     
Flag Coverage Δ
frontend 66.63% <66.66%> (+0.25%) ⬆️
python 79.48% <ø> (-0.01%) ⬇️
ruby-api 82.58% <ø> (+0.10%) ⬆️
ruby-backend 85.14% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 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.

@sonarqubecloud

Copy link
Copy Markdown

@ryanmelt
ryanmelt merged commit 6055c7a into main Sep 15, 2026
36 checks passed
@ryanmelt
ryanmelt deleted the immediate_screen_update branch September 15, 2026 03:04
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