Skip to content

Document ReportShared object removal in migration guide#177

Merged
dvacca-onfido merged 1 commit intomasterfrom
update-migration-guide
Mar 6, 2026
Merged

Document ReportShared object removal in migration guide#177
dvacca-onfido merged 1 commit intomasterfrom
update-migration-guide

Conversation

@dvacca-onfido
Copy link
Collaborator

@dvacca-onfido dvacca-onfido commented Mar 6, 2026

What Changed

Added missing documentation to the v6 → v7 migration guide for the ReportShared object removal.

Details

  • The ReportShared object has been dropped in v7
  • Convenience accessors (getName(), getStatus(), getId()) added to parent Report object
  • Other properties now accessed via specific report type accessors (e.g., getDocumentReport())

Migration Example

// Before (v6.x)
ReportName name = report.getReportShared().getName();
String checkId = report.getReportShared().getCheckId();

// After (v7.x)
ReportName name = report.getName(); // Convenience accessor
String checkId = report.getDocumentReport().getCheckId(); // Via specific type

- Add detailed migration instructions for ReportShared removal
- Clarify that name and status remain on parent Report object
- Show that other properties moved to specific report type accessors
- Include example showing document report accessor usage
@dvacca-onfido dvacca-onfido force-pushed the update-migration-guide branch from e0915ae to 9851555 Compare March 6, 2026 09:32
@dvacca-onfido dvacca-onfido changed the title Document ReportShared object removal in migration guide Update migration guide: Document ReportShared object removal Mar 6, 2026
@dvacca-onfido dvacca-onfido changed the title Update migration guide: Document ReportShared object removal Document ReportShared object removal in migration guide Mar 6, 2026
@dvacca-onfido dvacca-onfido merged commit c9620b5 into master Mar 6, 2026
6 of 7 checks passed
@dvacca-onfido dvacca-onfido deleted the update-migration-guide branch March 6, 2026 10:32
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.

3 participants