Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set user property when there is a static CSV and entity list clash #6466

Merged
merged 2 commits into from
Oct 23, 2024

Conversation

seadowg
Copy link
Member

@seadowg seadowg commented Oct 22, 2024

Closes #6455

As well as the case we're worried about this will also detect a couple that won't be a problem:

  1. A secondary instance CSV that has a name like people.csv in a project that has a people entity list, but the instance ID is not people and so is safe (static-people for example).
  2. A CSV that has a name like people.csv in a project that has a people entity list, but the CSV is not being used as the src for a secondary instance.

Both these cases should be very rare though. I've also updated the name of the user property we were using to track metered non-cellular connections as we'd forgotten to set the user property up on Firebase and the name we were using was too long (there is a 40-character limit).

@seadowg seadowg marked this pull request as ready for review October 22, 2024 15:59
@seadowg seadowg requested a review from lognaturel October 22, 2024 15:59
@seadowg seadowg merged commit 49c74d9 into getodk:master Oct 23, 2024
6 checks passed
@seadowg seadowg deleted the static-csv-property branch October 23, 2024 09:12
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.

Analytics: track user property for match between static CSV and Entity List names
2 participants