Skip to content

refactor: remove Certificate Management screen from RN verifier tutorial#287

Merged
avner-m merged 1 commit into
masterfrom
aa/remove-certificate-management-screen
Apr 30, 2026
Merged

refactor: remove Certificate Management screen from RN verifier tutorial#287
avner-m merged 1 commit into
masterfrom
aa/remove-certificate-management-screen

Conversation

@al-abbas-nz
Copy link
Copy Markdown
Contributor

Summary

  • Deletes the CertificateManagementModal screen from the completed RN mDocs Verifier tutorial app. The sample Montcliff DMV IACA is already registered automatically during SDK initialization (see docs: Added code annotations to RN completed app #282 / e5db924), so the manual add/view/remove UI is redundant.
  • Strips the "Certificate Management" button, showCertificateManagement / trustedCertificates state, unused imports, and the "No trusted issuer certificates added" error text from App.tsx so the complete app reflects the simplified flow.

This mirrors the direction Pavel is taking for the Android tutorial in mattr-learn#977; the companion mattr-learn doc changes for React Native have been pushed onto that branch.

Test plan

  • yarn install and run the completed app on Android — app launches, auto-loads the Montcliff IACA, and the home screen shows only the "Scan QR Code" button.
  • Verify an mDL presentation end-to-end still works (scan → consent → view response).
  • npx tsc --noEmit passes in react-native-mdocs-verifier-tutorial-complete/.

…ete app

The sample IACA certificate is registered automatically during SDK
initialization (see e5db924), so the manual Certificate Management
screen is no longer useful in the completed tutorial app. Delete
CertificateManagementModal.tsx and strip the corresponding button,
state, imports and "no trusted certificates" error text from App.tsx
so the completed app reflects the simplified flow.
@al-abbas-nz al-abbas-nz requested a review from a team as a code owner April 24, 2026 01:54
@avner-m avner-m merged commit e948008 into master Apr 30, 2026
2 checks passed
@avner-m avner-m deleted the aa/remove-certificate-management-screen branch April 30, 2026 04:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants