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 camera permission granted listener #234

Merged
merged 2 commits into from
Jan 7, 2025

Conversation

Fabtron
Copy link
Collaborator

@Fabtron Fabtron commented Jan 2, 2025

Add listener to SelfScanningFragment to react to granted camera permission, to get rid of deprecated features.

APPS-2047

How to test?

Integrate this branch into an app using the callback and see if it works.

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 requested a review from cmaier January 2, 2025 08:50
@Fabtron Fabtron marked this pull request as ready for review January 2, 2025 08:50
@Fabtron Fabtron merged commit ca54706 into main Jan 7, 2025
1 check passed
@Fabtron Fabtron deleted the add-permissiongranted-listener-to-selfscanning-view branch January 7, 2025 07:11
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