feat: enabling QR codes for users (WPB-12115)

[yamilmedina]

WPB-12115 [Android] : QR code opens up deeplinks which links directly to the app rather than going to the web

---

PR Submission Checklist for internal contributors

• The PR Title

  • conforms to the style of semantic commits messages¹ supported in Wire's Github Workflow²
  • contains a reference JIRA issue number like SQPIT-764
  • answers the question: If merged, this PR will: ... ³

• The PR Description

  • is free of optional paragraphs and you have filled the relevant parts to the best of your ability

---

What's new in this PR?

Issues

Previously we had to disable the QR code feature, to assess some last moment requirements. #3569
Now everything is sorted out, definitions done, so we need to modify the code and re-enable.

Solutions

This PR:

• Enables QR code by feature flag
• Implement deeplink according to new specs (direct deeplink, instead of accounts)
• Handles the same error dialog for other user profiles, as in iOS and Web

Testing

Test Coverage (Optional)

• I have added automated test to this contribution

---

PR Post Submission Checklist for internal contributors (Optional)

• Wire's Github Workflow has automatically linked the PR to a JIRA issue

---

PR Post Merge Checklist for internal contributors

• If any soft of configuration variable was introduced by this PR, it has been added to the relevant documents and the CI jobs have been updated.

---

References

1. https://sparkbox.com/foundry/semantic_commit_messages
2. https://github.com/wireapp/.github#usage
3. E.g. feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 41.17647% with 10 lines in your changes missing coverage. Please review.

Project coverage is 45.34%. Comparing base (e8b5148) to head (fd75a42).

Files with missing lines	Patch %	Lines
...erprofile/other/OtherUserProfileScreenViewModel.kt	0.00%	6 Missing ⚠️
...roid/ui/userprofile/other/OtherUserProfileState.kt	50.00%	2 Missing ⚠️
...om/wire/android/util/deeplink/DeepLinkProcessor.kt	33.33%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@                Coverage Diff                 @@
##           release/candidate    #3616   +/-   ##
==================================================
  Coverage              45.33%   45.34%           
==================================================
  Files                    471      471           
  Lines                  15835    15845   +10     
  Branches                2654     2655    +1     
==================================================
+ Hits                    7179     7185    +6     
- Misses                  7885     7887    +2     
- Partials                 771      773    +2     

Files with missing lines	Coverage Δ
.../wire/android/ui/userprofile/qr/SelfQRCodeState.kt	100.00% <100.00%> (ø)
...e/android/ui/userprofile/qr/SelfQRCodeViewModel.kt	50.00% <100.00%> (+1.35%)	⬆️
.../wire/android/util/debug/FeatureVisibilityFlags.kt	0.00% <ø> (ø)
...roid/ui/userprofile/other/OtherUserProfileState.kt	74.00% <50.00%> (-2.60%)	⬇️
...om/wire/android/util/deeplink/DeepLinkProcessor.kt	76.33% <33.33%> (+1.72%)	⬆️
...erprofile/other/OtherUserProfileScreenViewModel.kt	58.58% <0.00%> (-1.21%)	⬇️

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e8b5148...fd75a42. Read the comment docs.

[github-actions]

Built wire-android-staging-compat-pr-3616.apk is available for download

[github-actions]

Built wire-android-staging-compat-pr-3616.apk is available for download

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

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

See analysis details on SonarCloud

[github-actions]

Built wire-android-staging-compat-pr-3616.apk is available for download