Skip to content

Add experimental opt-in for entities spec v2025.1#6930

Merged
seadowg merged 5 commits intogetodk:masterfrom
grzesiek2010:COLLECT-6923
Oct 17, 2025
Merged

Add experimental opt-in for entities spec v2025.1#6930
seadowg merged 5 commits intogetodk:masterfrom
grzesiek2010:COLLECT-6923

Conversation

@grzesiek2010
Copy link
Member

@grzesiek2010 grzesiek2010 commented Oct 14, 2025

Closes #6923

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?

We need to just verify that forms that use entities with spec v2025.1 work when the new setting is enabled. Otherwise, there should be no change in behavior.

Do we need any specific form for testing your changes? If so, please attach one.

Forms that use entities with spec v2025.1

Does this change require updates to documentation? If so, please file an issue here and include the link below.

No.

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

@grzesiek2010 grzesiek2010 marked this pull request as ready for review October 14, 2025 11:35
@grzesiek2010 grzesiek2010 requested a review from seadowg October 14, 2025 11:35
@grzesiek2010 grzesiek2010 requested a review from seadowg October 17, 2025 09:28
@seadowg seadowg merged commit 9ff6f3f into getodk:master Oct 17, 2025
6 checks passed
@dbemke
Copy link

dbemke commented Oct 21, 2025

I'm not sure about this case. When I download forms with entities spec v 20205.1, enable spec v2025.1 in Collect settings and open one of the form, it is possible to open the form even if I disable spec v2025.1 in settings afterwards. Is it ok for the time being?

Steps to reproduce:

  1. Download user "entInRepeat" https://staging.getodk.cloud/projects/97/app-users
  2. Enable spec 2025.1 in settings in Collect.
  3. Open one of the forms.
  4. Disable spec 2025.1 in settings in Collect.
  5. Try to open 3 forms ( the one opened before opens, other 2 show error about spec)

@dbemke
Copy link

dbemke commented Oct 21, 2025

Should forms with spec 2025.1 and the setting enabled in Collect allow creating offline entities in update forms with entities in repeats?

@seadowg
Copy link
Member

seadowg commented Oct 21, 2025

I'm not sure about this case. When I download forms with entities spec v 20205.1, enable spec v2025.1 in Collect settings and open one of the form, it is possible to open the form even if I disable spec v2025.1 in settings afterwards. Is it ok for the time being?

Ah interesting. I guess that's because it checks when parsing so once the cached form definition is available it won't check again. I think that's fine for now.

Should forms with spec 2025.1 and the setting enabled in Collect allow creating offline entities in update forms with entities in repeats?

No. We haven't implemented offline entities from repeats yet. Forms should behave as if they were using the v2024.1 spec.

@WKobus
Copy link

WKobus commented Oct 22, 2025

Tested with success

Verified on Android 16

Verified cases:

  • Regular entity forms and entity forms with repeat with v2025.1 spec
  • Mixing different specs e.g. register entity on v2024.1 updating same entity on v2025.1
  • Forms with spec older than 2025.1
  • Enable v2025.1 setting on and off

@dbemke
Copy link

dbemke commented Oct 22, 2025

Tested with success

Verified on Android 10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add experimental opt-in for entities spec v2025.1

4 participants