fix(api): Ensure Configure Nozzle Layout and Configure For Volume commands publish notifications #18631
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
Related to debugging work on RQA-4267
It seems we missed adding publishers for Configure Nozzle Layout and Configure For Volume. This made debugging and figuring out what kind of partial tip setup the pipette was actually in while going through logs a royal pain in the neck. Whoopsies!
Test Plan and Hands on Testing
Changelog
Review requests
We're missing a few more of these, like on
detect_liquid_presence
,require_liquid_presence
andmeasure_liquid_height
as well. Thats on instrument context, we could be missing others on protocol context and module context. Should I scrape around and add other missing ones? This PR was originally just to help to debug an ongoing RQA ticket, but could be for general sustaining work as well.Risk assessment
Low, just adding some missing publishers.