ryanw-mobile / OctoMeter

🔥Kotlin Multiplatform Desktop/Android/iOS Energy Tracker app
Other
78 stars 7 forks source link

AccountScreen: fix to correctly display a variable rate tariff #249

Closed ryanw-mobile closed 6 days ago

ryanw-mobile commented 6 days ago

We made the when sequence incorrect so that a tariff with a variable rate could never be detected and shown using the correct layout.

A quick fix to rearrange the sequence so the variable rate will be checked first.

codacy-production[bot] commented 6 days ago

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
:white_check_mark: -0.03% (target: -1.00%) :white_check_mark: ∅
Coverage variation details | | Coverable lines | Covered lines | Coverage | | ------------- | ------------- | ------------- | ------------- | | Common ancestor commit (c3a9dca82164fd498c977223a2f0499a199c567d) | 2129 | 1398 | 65.66% | | | Head commit (016a707953fbdd8dcba0921ed1972976b4a4fbae) | 2130 (+1) | 1398 (+0) | 65.63% (**-0.03%**) | **Coverage variation** is the difference between the coverage for the head and common ancestor commits of the pull request branch: ` - `
Diff coverage details | | Coverable lines | Covered lines | Diff coverage | | ------------- | ------------- | ------------- | ------------- | | Pull request (#249) | 0 | 0 | **∅ (not applicable)** | **Diff coverage** is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: `/ * 100%`

See your quality gate settings    Change summary preferences


:rocket: Don’t miss a bit, follow what’s new on Codacy.

Codacy stopped sending the deprecated coverage status on June 5th, 2024. Learn more