Skip to content

Comments

SIMSBIOHUB-870: Align payload property names with BioHub#1608

Merged
dylanrogowsky-oxd merged 2 commits intodevfrom
SIMSBIOHUB-870
Feb 12, 2026
Merged

SIMSBIOHUB-870: Align payload property names with BioHub#1608
dylanrogowsky-oxd merged 2 commits intodevfrom
SIMSBIOHUB-870

Conversation

@dylanrogowsky-oxd
Copy link
Collaborator

@dylanrogowsky-oxd dylanrogowsky-oxd commented Feb 11, 2026

Links to Jira Tickets

Description of Changes

  • observation_environmental_condition (quantitative): Use environmental_condition_value for quantitative conditions instead of environmental_quantitative_value, so both qualitative and quantitative use the same property name and match what the companion system expects.
  • telemetry_device: Renamed payload property from model to device_model to match the companion audit.
  • report (report attachments): Send artifact_key (from report attachment key), name (from title), and year (from year_published) instead of artifact_id, title, and year_published so the payload matches the companion audit.
  • Updated the tests that were asserting on the old property names so the suite passes.

Testing Notes

  • npm test in the api package — all tests pass. The only changes in this PR are the payload property names above and the corresponding test expectations; no behaviour change beyond what we send to the API.

@dylanrogowsky-oxd dylanrogowsky-oxd changed the title SIMSBIOHUB-870: Align BioHub create payload property names with companion system SIMSBIOHUB-870: Align payload property names with BioHub Feb 11, 2026
@codecov
Copy link

codecov bot commented Feb 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 33.50%. Comparing base (7e65466) to head (1604b4f).
⚠️ Report is 1 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #1608      +/-   ##
==========================================
+ Coverage   33.05%   33.50%   +0.44%     
==========================================
  Files        1034     1035       +1     
  Lines       65196    65951     +755     
  Branches     2357     2490     +133     
==========================================
+ Hits        21553    22097     +544     
- Misses      43103    43270     +167     
- Partials      540      584      +44     

☔ 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.

@dylanrogowsky-oxd dylanrogowsky-oxd added Do Not Merge PR should not be merged Early Feedback Welcome PR is not finished, but early review feedback is welcomed labels Feb 11, 2026
@sonarqubecloud
Copy link

@dylanrogowsky-oxd dylanrogowsky-oxd added Ready For Review PR is ready for review Early Feedback Welcome PR is not finished, but early review feedback is welcomed and removed Early Feedback Welcome PR is not finished, but early review feedback is welcomed Ready For Review PR is ready for review Do Not Merge PR should not be merged labels Feb 11, 2026
Copy link
Collaborator

@mauberti-bc mauberti-bc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@dylanrogowsky-oxd dylanrogowsky-oxd marked this pull request as ready for review February 12, 2026 21:30
@dylanrogowsky-oxd dylanrogowsky-oxd merged commit f7d9bbf into dev Feb 12, 2026
36 checks passed
@dylanrogowsky-oxd dylanrogowsky-oxd deleted the SIMSBIOHUB-870 branch February 12, 2026 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ready For Review PR is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants