Skip to content

Conversation

@koukarine
Copy link
Contributor

Move multipaz-models code to the multipaz module.

Test: All Unit tests are passing (bar Longfellow).
Test: Manually tested via samples/testapp on Android.

Fixes #1334

  • Tests pass

@davidz25
Copy link
Contributor

I would also drop the .models from the package names except for org.multipaz.models.presentment.* which should be changed to org.multipaz.presentment.model.*.

@koukarine koukarine force-pushed the models_merge branch 5 times, most recently from c37b159 to 2c653a0 Compare October 28, 2025 18:54
@koukarine koukarine requested a review from davidz25 October 28, 2025 19:02
@koukarine koukarine force-pushed the models_merge branch 2 times, most recently from 48cb8b6 to 1095a1c Compare October 28, 2025 21:41
@koukarine koukarine marked this pull request as ready for review October 28, 2025 22:26
@koukarine koukarine force-pushed the models_merge branch 2 times, most recently from 845b02b to e5bd684 Compare October 29, 2025 03:59
Test: All Unit tests are passing (bar Longfellow).
Test: Manually tested via samples/testapp on Android.

Signed-off-by: koukarine <[email protected]>
@davidz25 davidz25 merged commit 9b02226 into openwallet-foundation:main Oct 29, 2025
3 checks passed
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.

Move module multipaz-models into multipaz as a package.

2 participants