Skip to content

feat: Add support for Bonus PARI credential#203

Open
NikkingSpin wants to merge 11 commits into
WLS-99from
WLS-122-pari-credential
Open

feat: Add support for Bonus PARI credential#203
NikkingSpin wants to merge 11 commits into
WLS-99from
WLS-122-pari-credential

Conversation

@NikkingSpin
Copy link
Copy Markdown
Collaborator

@NikkingSpin NikkingSpin commented May 26, 2026 •
edited by atlassian Bot
Loading

Short description

Add support for the Bonus PARI credential with a new card component, barcode integration, and related utilities.

List of changes proposed in this pull request

  • Add Bonus PARI credential with new BonusCard component
  • Add barcode library (react-native-barcode-builder) and related patch
  • Add bonus card asset images
  • Add ItwBarcodeCard component for barcode display in credential details
  • Update credential claims and presentation components to support the new bonus card
  • Add useDetectSmallScreen and useScreenReaderEnabled hooks
  • Update itwStyleUtils and itwTypesUtils for the new credential configuration
  • Update wallet locales with bonus-related strings

How to test

  1. Checkout the branch and run pnpm install
  2. Complete the credential issuance flow to obtain a Bonus PARI credential
  3. Navigate to the credential details screen and verify the Bonus PARI details renders correctly with its barcode and status

Resolves WLS-122

@NikkingSpin NikkingSpin changed the title Wls 122 pari credential feat: Add support for Bonus PARI credential May 26, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 26, 2026

PR Title Validation for conventional commit type

✅ All good! PR title follows the conventional commit type.

sebastianopriscan
sebastianopriscan requested changes May 27, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@sebastianopriscan sebastianopriscan left a comment

Choose a reason for hiding this comment

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

There seem to be issues during the Android build process.

@sebastianopriscan
Copy link
Copy Markdown
Collaborator

sebastianopriscan commented May 27, 2026

@LazyAfternoons, should this PR be merged into main instead of WLS-99, and add in its description the "Depends on" (which I think is not so useful because of dpulls not being available) ? This way a review would be required for merging and, in case WLS-99 is approved, squashed and deleted, the PR would still be valid.

sebastianopriscan
sebastianopriscan reviewed May 27, 2026
View reviewed changes
Comment thread libs/it-wallet/src/assets/img/credential/bonus_pari.png
Copy link
Copy Markdown
Collaborator

@sebastianopriscan sebastianopriscan May 27, 2026

Choose a reason for hiding this comment

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

Is this asset used anywhere in the app? If not, could you please remove it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sebastianopriscan sebastianopriscan sebastianopriscan requested changes

@LazyAfternoons LazyAfternoons Awaiting requested review from LazyAfternoons

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@NikkingSpin @sebastianopriscan @LazyAfternoons