Feat reduce ingest visual noise#1777
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
51e8dba to
b2a2225
Compare
About the Contributor
This pull request is posted on behalf of the BBC.
Type of Contribution
This is a: Code improvement
Bug fix / Feature / Code improvement / Documentation improvement / Other (please specify)
Current Behavior
Ingest can be visually noisy with the current part instance flickering as other segments / parts update
New Behavior
Noise is reduced
Additional feature added to allow blueprints to request
syncIngestUpdateToPartInstancebe explicitly ran at ingest, important for runtime operations such as T-Timers and prevents the need to add data to every part (as there is no way to know what the active part is).Testing
Affected areas
This PR affects the webui and ingest logic
Time Frame
Not urgent, but we would like to get this merged into the in-development release.
Other Information
Written with the assistance of copilot and undergoing user tests before bringing out of draft.
Status