Skip to content

Add analytics for no-buttons columns-n#6940

Merged
grzesiek2010 merged 3 commits intogetodk:v2025.3.xfrom
seadowg:columns-analytics
Oct 29, 2025
Merged

Add analytics for no-buttons columns-n#6940
grzesiek2010 merged 3 commits intogetodk:v2025.3.xfrom
seadowg:columns-analytics

Conversation

@seadowg
Copy link
Member

@seadowg seadowg commented Oct 17, 2025

This adds analytics so we can get insight into how many people are currently using the "gallery" appearance for selects. This will hopefully let us make more informed decisions when discussing #6920.

Why is this the best possible solution? Were any other approaches considered?

Nothing to discuss here!

How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?

No testing needed.

Before submitting this PR, please make sure you have:

  • added or modified tests for any new or changed behavior
  • run ./gradlew connectedAndroidTest (or ./gradlew testLab) and confirmed all checks still pass
  • added a comment above any new strings describing it for translators
  • added any new strings with date formatting to DateFormatsTest
  • verified that any code or assets from external sources are properly credited in comments and/or in the about file.
  • verified that any new UI elements use theme colors. UI Components Style guidelines

@seadowg seadowg changed the base branch from master to v2025.3.x October 17, 2025 12:55
@seadowg seadowg marked this pull request as ready for review October 17, 2025 13:25
@seadowg seadowg requested review from grzesiek2010 and lognaturel and removed request for grzesiek2010 October 17, 2025 13:25
@seadowg seadowg requested review from grzesiek2010 and removed request for lognaturel October 28, 2025 14:11
@seadowg seadowg requested a review from grzesiek2010 October 29, 2025 09:25
const val ADD_SHORTCUT = "AddShortcut"

/**
* Tracks how many forms include the `no-buttons columns-n` appearance
Copy link
Member

Choose a reason for hiding this comment

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

columns-n or columns

@grzesiek2010 grzesiek2010 self-requested a review October 29, 2025 16:53
@grzesiek2010 grzesiek2010 merged commit b58c489 into getodk:v2025.3.x Oct 29, 2025
6 checks passed
@seadowg seadowg deleted the columns-analytics branch October 29, 2025 17:55
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.

3 participants