openwallet-foundation / bifold-wallet

Aries Mobile Agent React Native - Part of the Aries Bifold effort to provide SSI capabilities in a production ready app.
Apache License 2.0
152 stars 149 forks source link

feat: add injectable stack nav, re-add theming #1289

Closed bryce-mcmath closed 1 month ago

bryce-mcmath commented 1 month ago

Summary of Changes

This PR allows Bifold wallets to extend the RootStack with an optional custom stack navigator slot. It also reverts a recent change that remove theming from the PINInput.

Related Issues

N/A

Pull Request Checklist

Tick all boxes below to demonstrate that you have completed the respective task. If the item does not apply to your this PR check it anyway to make it apparent that there's nothing to do.

If you have any questions to any of the points above, just submit and ask! This checklist is here to help you, not to deter you from contributing!

Pro Tip 🤓

PR template adapted from the Python attrs project.

sonarcloud[bot] commented 1 month ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

bryce-mcmath commented 1 month ago

@FarjadGov14 we had to add back the PINInputTheme.cell usage, as it broke the appearance in BC Wallet. You can edit this property in your theme.ts file to make it look however you like, but the theming should still be used so that Bifold users can customize it.