Skip to content

Remove "Buy sensors" links#1540

Closed
ojousima wants to merge 64 commits intomasterfrom
issue-1532-remove-buy-sensors-links
Closed

Remove "Buy sensors" links#1540
ojousima wants to merge 64 commits intomasterfrom
issue-1532-remove-buy-sensors-links

Conversation

@ojousima
Copy link
Member

@ojousima ojousima commented Mar 3, 2026

closes #1532

Summary

  • remove "Buy Ruuvi sensors" item from the main drawer menu (Compose and legacy XML menu)
  • remove "Buy Ruuvi Sensors" buttons from Add Sensor screen
  • remove click handlers that opened buy-sensors URLs

Review notes

  • localization string files are generated via scripts/ci/update_l10n.sh, so this PR intentionally avoids manual string-key deletions
  • verified compile/resources with:
    • ./gradlew :app:compileWithoutFileLogsDebugKotlin :app:mergeWithoutFileLogsDebugResources
  • no behavior changes beyond removing the buy-sensors entry points

andreevdenis and others added 30 commits December 23, 2025 00:02
When zooming/panning charts, the rightmost time label would sometimes
disappear even when there was space to display it. This was caused by
integer division rounding down when calculating the last label position.

Changed lastPoint calculation to use ceil() to ensure labels extending
to the visible edge are always included.
…labels

Fix missing X-axis labels at chart edge
…ation into milestone/3.4

# Conflicts:
#	app/build.gradle
andreevdenis and others added 26 commits February 13, 2026 19:36
@ojousima
Copy link
Member Author

ojousima commented Mar 3, 2026

Wrong base branch, closing

@ojousima ojousima closed this Mar 3, 2026
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 3, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
20 Security Hotspots
0.1% Coverage on New Code (required ≥ 80%)
E Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@ojousima ojousima deleted the issue-1532-remove-buy-sensors-links branch March 3, 2026 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove "Buy sensors" links

3 participants