Skip to content

MS-1037 Also count enrolment v4 for record upload counter in sync info#1239

Merged
luhmirin-s merged 1 commit into
release/2025.2.0-draftfrom
feature/MS-1037-upsync-record-counter-fix
Jun 26, 2025
Merged

MS-1037 Also count enrolment v4 for record upload counter in sync info#1239
luhmirin-s merged 1 commit into
release/2025.2.0-draftfrom
feature/MS-1037-upsync-record-counter-fix

Conversation

@luhmirin-s
Copy link
Copy Markdown
Contributor

JIRA ticket
Will be released in: 2025.2.0

Root cause analysis (for bugfixes only)

First known affected version: 2025.2.0

  • When implementing record update we added a new schema version of enrolment events, but did not add the new type to be counted for upload.

Notable changes

  • Count both types of enrolment event versions in the sync info screen.
  • Simplify the count method for "all" events by removing the optional type argument that is never provided. If it is needed in the future, the new list argument version can be used instead.

Testing guidance

  • Do a couple of enrolments (without sync)
  • Check sync info screen, the "Records to Upload" should not be 0.

Additional work checklist

  • Effect on other features and security has been considered
  • Design document marked as "In development" (if applicable)
  • External (Gitbook) and internal (Confluence) Documentation is up to date (or ticket created)
  • Test cases in Testiny are up to date (or ticket created)
  • Other teams notified about the changes (if applicable)

@luhmirin-s luhmirin-s requested review from a team, BurningAXE, TristramN, alex-vt, alexandr-simprints, meladRaouf and ybourgery and removed request for a team June 26, 2025 07:09
@cla-bot cla-bot Bot added the ... label Jun 26, 2025
@sonarqubecloud
Copy link
Copy Markdown

@luhmirin-s luhmirin-s merged commit 54c9ebc into release/2025.2.0-draft Jun 26, 2025
20 of 21 checks passed
@luhmirin-s luhmirin-s deleted the feature/MS-1037-upsync-record-counter-fix branch June 26, 2025 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants