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

App usages over 16 years (EXPOSUREAPP-1781) #1048

Merged
merged 6 commits into from
Aug 26, 2020
Merged

Conversation

apopovsap
Copy link
Contributor

@apopovsap apopovsap requested a review from a team August 20, 2020 11:42
@kolyaopahle kolyaopahle added the maintainers Tag pull requests created by maintainers label Aug 20, 2020
@jakobmoellerdev jakobmoellerdev changed the title Exposureapp 1781 App usages over 16 years (EXPOSUREAPP-1781) Aug 20, 2020
@kolyaopahle kolyaopahle added the ui Issue related to UI aspects label Aug 20, 2020
Copy link
Contributor

@kolyaopahle kolyaopahle left a comment

Choose a reason for hiding this comment

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

LGTM 👍 i made some small linting suggestions other than those im quite happy


<variable
name="tracingViewModel"
type="de.rki.coronawarnapp.ui.viewmodel.TracingViewModel" />
Copy link
Contributor

Choose a reason for hiding this comment

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

Variable seems unused


<data>

<import type="de.rki.coronawarnapp.util.formatter.FormatterHelper" />
Copy link
Contributor

Choose a reason for hiding this comment

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

Formatters seem unused


<import type="de.rki.coronawarnapp.util.formatter.FormatterHelper" />

<import type="de.rki.coronawarnapp.util.formatter.FormatterAccessibilityHelper" />
Copy link
Contributor

Choose a reason for hiding this comment

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

Formatters seem unused


<import type="de.rki.coronawarnapp.util.formatter.FormatterAccessibilityHelper" />

<import type="android.view.View" />
Copy link
Contributor

Choose a reason for hiding this comment

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

View seems unused

###################################### -->

<!-- XACT: onboarding(sixteen) title -->
<string name="sixteen_title_text">"App Nutzung ab 16 Jahren"</string>
Copy link
Contributor

Choose a reason for hiding this comment

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

Default strings should be empty

<string name="sixteen_title_text">"App Nutzung ab 16 Jahren"</string>

<!-- XACT: onboarding(sixteen) title -->
<string name="sixteen_description_text">"Die App richtet sich an Personen, die sich in Deutschland aufhalten und mindestens 16 Jahre alt sind."</string>
Copy link
Contributor

Choose a reason for hiding this comment

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

Default strings should be empty

@@ -198,6 +205,14 @@
<item name="android:textColor">@color/colorTextPrimary1</item>
</style>

<style name="headline6Sixteen" parent="@style/TextAppearance.MaterialComponents.Headline6">
<item name="android:textColor">@color/colorTextEmphasizedButton</item>
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a non button reference you could use?

</style>

<style name="subtitleSixteen" parent="@style/TextAppearance.MaterialComponents.Subtitle1">
<item name="android:textColor">@color/colorTextEmphasizedButton</item>
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a non button reference you could use?

@apopovsap
Copy link
Contributor Author

fixed

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@jakobmoellerdev jakobmoellerdev merged commit 3b4a559 into dev Aug 26, 2020
@jakobmoellerdev jakobmoellerdev deleted the EXPOSUREAPP_1781 branch August 26, 2020 11:28
@jakobmoellerdev
Copy link
Contributor

jakobmoellerdev commented Aug 27, 2020

EXPOSUREAPP-2092

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 sprint5 ui Issue related to UI aspects
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants