Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add deposit return voucher states as part of the checkout process (Apps-2040) #233

Merged

Conversation

Fabtron
Copy link
Collaborator

@Fabtron Fabtron commented Dec 27, 2024

What's new?

We can now track the state of deposit return vouchers in the checkout process. If a redemption failes, due to whatever reason, we receive a matching state in a new property depositReturnVouchers which holds the state for each drv. The state is handled and the user is informed via a Dialog that the reffering deposit return voucher will be removed to proceed with the checkout, e.g. to try again without it.

APPS-2040

How to test?

Integrate this branch into an app using drv's and scan the product linked in the issue. Add an additional item to get to a positive amount and checkout. -> The dialog should pop up after the checkout failed.

Definition of Done

  • Issue is linked
  • All requirements of the issue are fulfilled
  • Changelog is updated
  • Documentation is updated
  • Self-Review
  • Review with the Product Owner (Release-Variante o. Minified Build)

App Tests

  • Minified build has been tested (aka. Release Build)
  • Environments have been taken care of (Production/Staging)
  • Supported languages have been tested
  • Light-/Dark-Mode has been tested
  • Edge-Cases have been tested
  • Android API Levels have been taken care of (minSdk?)

Testing

  • Tests have been written (aka Unit-Tests, Integration-Tests, ...)

@Fabtron Fabtron marked this pull request as ready for review December 30, 2024 09:56
@Fabtron Fabtron requested a review from cmaier December 30, 2024 09:56
…into apps-2040-add-drv-to-checkout-process

# Conflicts:
#	CHANGELOG.md
#	ui/src/main/res/values-de/strings.xml
#	ui/src/main/res/values/strings.xml
Copy link
Contributor

@cmaier cmaier left a comment

Choose a reason for hiding this comment

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

Works fine. Just minor findings.

@Fabtron Fabtron requested a review from cmaier January 7, 2025 07:11
Fabtron and others added 2 commits January 7, 2025 14:27
…into apps-2040-add-drv-to-checkout-process

# Conflicts:
#	CHANGELOG.md
#	core/src/main/java/io/snabble/sdk/shoppingcart/ShoppingCart.kt
#	ui/src/main/res/values/strings.xml
@Fabtron Fabtron merged commit e9fe1af into feature-deposit-return-voucher Jan 7, 2025
1 check passed
@Fabtron Fabtron deleted the apps-2040-add-drv-to-checkout-process branch January 7, 2025 16:15
@Fabtron Fabtron mentioned this pull request Jan 15, 2025
13 tasks
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.

2 participants