Skip to content

AQI-10938: Add REPORT_RUN_STATUS_FORM_SUBMITTED#132

Merged
aaron-jarrette merged 4 commits into
mainfrom
feature/AQI-10938
May 26, 2026
Merged

AQI-10938: Add REPORT_RUN_STATUS_FORM_SUBMITTED#132
aaron-jarrette merged 4 commits into
mainfrom
feature/AQI-10938

Conversation

@mathitharmalingam-aqi
Copy link
Copy Markdown
Contributor

No description provided.

@mathitharmalingam-aqi mathitharmalingam-aqi requested a review from a team May 26, 2026 18:16
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR bumps the ONE.Models.CSharp package version to 7.23.0 and updates the changelog to reflect a new “added” item.

Changes:

  • Bumped VersionPrefix from 7.22.0 to 7.23.0 in the C# models project.
  • Added a 7.23.0 entry to CHANGELOG.md noting REPORT_RUN_STATUS_FORM_SUBMITTED.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
src/ONE.Models.CSharp/ONE.Models.CSharp.csproj Increments package version to 7.23.0 (also introduces a UTF-8 BOM on the first line).
CHANGELOG.md Adds a 7.23.0 changelog entry (but currently claims a value was added without corresponding code/proto changes).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@@ -1,9 +1,9 @@
<Project Sdk="Microsoft.NET.Sdk">
<Project Sdk="Microsoft.NET.Sdk">
Comment thread CHANGELOG.md Outdated
Comment on lines +5 to +9
## Version 7.23.0 - 2026-05-25

### Added

- Add REPORT_RUN_STATUS_FORM_SUBMITTED (AQI-10938)
Comment thread CHANGELOG.md
Comment on lines +9 to +11
- Add REPORT_RUN_STATUS_FORM_SUBMITTED (AQI-10938)


AQI-DanG
AQI-DanG previously approved these changes May 26, 2026
Copy link
Copy Markdown
Contributor

@MatthewDeRose-AI MatthewDeRose-AI left a comment

Choose a reason for hiding this comment

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

I think the instrument health group has already claimed 7.23.0, I saw they had a PR openhttps://github.com//pull/130

@aaron-jarrette aaron-jarrette merged commit 91f7c80 into main May 26, 2026
1 check passed
@aaron-jarrette aaron-jarrette deleted the feature/AQI-10938 branch May 26, 2026 19:41
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.

5 participants