Skip to content

[MCC-1563432] Python Lib - Return blinding status and warnings from dry_publish - #70

Merged
afieraru-mdsol merged 1 commit into
mainfrom
MCC_1563432_dry_publish_blinding_status
Oct 19, 2026
Merged

afieraru-mdsol merged 1 commit into
mainfrom
MCC_1563432_dry_publish_blinding_status

Conversation

@snpawar-mdsol

@snpawar-mdsol snpawar-mdsol commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

feature/bug/enhance/refactor/tech_debt/hotfix

  • Python Lib - Return blinding status and warnings from dry_publish
  • MCC-1563432

Checklist

  • Review the pull request to fix typos and ensure variable/function names are intuitive, etc.
  • Make sure the pull request does not have excessive number of unnecessary commits. Utilize the git commit --amend --no-edit command to reduce commit messages when making small file changes (like changing linespacing).
  • Make sure you have added unit tests for the code changes. Tests should be added in the tests/ folder.
  • Modify docs, if required. Add any new documentation in the doc/ folder.
  • Rebase on latest active development branch (develop/main).

Changes Summary

Return blinding status and warnings from dry_publish

@snpawar-mdsol snpawar-mdsol changed the title MCC_1563432: Return blinding_status and warnings from dry_publish MCC_1563432: Python Lib - Return blinding status and warnings from dry_publish Oct 16, 2026
@snpawar-mdsol snpawar-mdsol self-assigned this Oct 16, 2026
@ibaig-mdsol ibaig-mdsol changed the title MCC_1563432: Python Lib - Return blinding status and warnings from dry_publish [MCC-1563432] Python Lib - Return blinding status and warnings from dry_publish Oct 19, 2026
@ibaig-mdsol
ibaig-mdsol requested a lite review from Copilot October 19, 2026 01:06

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.

🟢 Approval recommended

No unresolved issues were identified.

No open issues 🎉

What changed in this PR

Adds blinding status and warning propagation to dry_publish responses while preserving legacy defaults.

Changes:

  • Extends transport and domain result models.
  • Maps new fields through the service layer.
  • Adds parsing, mapping, and default-value tests.
File Description
tests/​test_dry_publish.py Tests parsing, mapping, and defaults.
dataconnect/​transport/​models.py Parses new response fields.
dataconnect/​service/​mappers.py Maps fields to domain results.
dataconnect/​models.py Exposes fields on publish results.

Review effort: Lite

@afieraru-mdsol
afieraru-mdsol merged commit 0a46914 into main Oct 19, 2026
5 checks passed
@afieraru-mdsol
afieraru-mdsol deleted the MCC_1563432_dry_publish_blinding_status branch October 19, 2026 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants