Skip to content
This repository has been archived by the owner on Jan 31, 2024. It is now read-only.

Jwen/qr code generator #227

Merged
merged 20 commits into from
May 2, 2022
Merged

Jwen/qr code generator #227

merged 20 commits into from
May 2, 2022

Conversation

jiabaow
Copy link
Collaborator

@jiabaow jiabaow commented May 1, 2022

ganerate qr code of the uid of user
Screen Shot 2022-05-01 at 19 35 30

}

private fun showQrCode(boolean: Boolean) {
findViewById<ConstraintLayout>(R.id.profileContent).visibility = if (boolean) GONE else VISIBLE
Copy link

Choose a reason for hiding this comment

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

Similar blocks of code found in 2 locations. Consider refactoring.


private fun showQrCode(boolean: Boolean) {
findViewById<ConstraintLayout>(R.id.profileContent).visibility = if (boolean) GONE else VISIBLE
findViewById<ConstraintLayout>(R.id.QrCodePage).visibility = if (boolean) VISIBLE else GONE
Copy link

Choose a reason for hiding this comment

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

Similar blocks of code found in 2 locations. Consider refactoring.

@jiabaow jiabaow marked this pull request as ready for review May 1, 2022 17:34
@jiabaow jiabaow self-assigned this May 1, 2022
Copy link
Collaborator

@laurislopata laurislopata left a comment

Choose a reason for hiding this comment

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

image
The code looks good, just one question. Do we need all of these different logo files?

@jiabaow
Copy link
Collaborator Author

jiabaow commented May 1, 2022

image The code looks good, just one question. Do we need all of these different logo files?

These are generated automatically by Android Studios to adapt to different use cases. I don't use them in this PR but I saw it is better to use mipmap to store the logo (https://stackoverflow.com/questions/28065267/mipmaps-vs-drawable-folders) which can be done in a separate PR.

@laurislopata
Copy link
Collaborator

image The code looks good, just one question. Do we need all of these different logo files?

These are generated automatically by Android Studios to adapt to different use cases. I don't use them in this PR but I saw it is better to use mipmap to store the logo (https://stackoverflow.com/questions/28065267/mipmaps-vs-drawable-folders) which can be done in a separate PR.

Okey, in that case LGTM

Copy link
Owner

@MaximeZmt MaximeZmt left a comment

Choose a reason for hiding this comment

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

Nice work 👍 Just left two comments below

@jiabaow jiabaow linked an issue May 2, 2022 that may be closed by this pull request
Copy link
Owner

@MaximeZmt MaximeZmt left a comment

Choose a reason for hiding this comment

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

😀👍

@codeclimate
Copy link

codeclimate bot commented May 2, 2022

Code Climate has analyzed commit f3dca7e and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 96.0% (80% is the threshold).

This pull request will bring the total coverage in the repository to 85.6% (0.2% change).

View more on Code Climate.

@jiabaow jiabaow merged commit 80c6e1b into main May 2, 2022
@jiabaow jiabaow deleted the jwen/qr-code-generator branch May 2, 2022 12:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

QR code generator for each user (#220)
3 participants