Skip to content

AQI-11674: Add PropertyBag to ReportDefinitionRun - #147

Merged
aaron-jarrette merged 5 commits into
mainfrom
feature/AQI-11674
Oct 13, 2026
Merged

aaron-jarrette merged 5 commits into
mainfrom
feature/AQI-11674

Conversation

@mathitharmalingam-aqi

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The protobuf source supplying the documented property is not updated.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 1 High severity

Open (1)
What changed in this PR

Prepares release 7.36.0 for the intended ReportDefinitionRun.PropertyBag addition, but omits the required protobuf submodule update.

Changes:

  • Bumps package version to 7.36.0.
  • Adds a release-note entry.
File Description
ONE.Models.CSharp.csproj Updates package version.
CHANGELOG.md Documents the intended property addition.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread CHANGELOG.md

### Added

- Added nullable `PropertyBag` (`StringValue`, protobuf field 11) to `ReportDefinitionRun`.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The submodule is pointing to a commit with PropertyBag.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

The pinned protobuf definition lacks the documented PropertyBag field.

Review effort: Balanced
Findings: 1 High severity

Open (1)

Base automatically changed from feature/AQI-11668 to main September 30, 2026 12:54
AQI-DanG
AQI-DanG previously approved these changes Oct 13, 2026
@mathitharmalingam-aqi
mathitharmalingam-aqi marked this pull request as ready for review October 13, 2026 21:48
@mathitharmalingam-aqi
mathitharmalingam-aqi requested a review from a team October 13, 2026 21:48
@aaron-jarrette
aaron-jarrette merged commit 77561da into main Oct 13, 2026
1 check passed
@aaron-jarrette
aaron-jarrette deleted the feature/AQI-11674 branch October 13, 2026 21:57
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