Skip to content
This repository has been archived by the owner on Jun 20, 2023. It is now read-only.

Profile screen (EXPOSUREAPP-6029) #2964

Merged
merged 69 commits into from
Apr 29, 2021

Conversation

mtwalli
Copy link
Contributor

@mtwalli mtwalli commented Apr 26, 2021

  • Creates Profile Qr Code screen
  • Simple V-Card format string replacement

TODO

  • generate Qr code from v-card format
  • Add some unit tests

@mtwalli mtwalli marked this pull request as ready for review April 28, 2021 14:04
@mtwalli mtwalli requested review from a team April 28, 2021 14:04
janetback
janetback previously approved these changes Apr 28, 2021
Copy link
Contributor

@janetback janetback left a comment

Choose a reason for hiding this comment

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

UA reviewed

@axelherbstreith axelherbstreith self-assigned this Apr 28, 2021
@AlexanderAlferov AlexanderAlferov self-assigned this Apr 29, 2021
Copy link
Contributor

@AlexanderAlferov AlexanderAlferov left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@sonarcloud
Copy link

sonarcloud bot commented Apr 29, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

21.2% 21.2% Coverage
0.0% 0.0% Duplication

Copy link
Contributor

@axelherbstreith axelherbstreith left a comment

Choose a reason for hiding this comment

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

Looks really nice! Great work 👍

@harambasicluka harambasicluka merged commit 2267bfb into release/2.2.x Apr 29, 2021
@harambasicluka harambasicluka deleted the feature/6029-profile-screen branch April 29, 2021 12:31
Copy link
Contributor

@janetback janetback left a comment

Choose a reason for hiding this comment

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

UA reviewed

Copy link
Contributor

@janetback janetback left a comment

Choose a reason for hiding this comment

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

UA reviewed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
maintainers Tag pull requests created by maintainers text change PRs with text changes. ui Issue related to UI aspects
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants