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

Changing language/theme during form entry blocks both match exactly and opening the form #6482

Open
dbemke opened this issue Oct 30, 2024 · 10 comments · May be fixed by #6488
Open

Changing language/theme during form entry blocks both match exactly and opening the form #6482

dbemke opened this issue Oct 30, 2024 · 10 comments · May be fixed by #6488
Assignees
Milestone

Comments

@dbemke
Copy link

dbemke commented Oct 30, 2024

ODK Collect version

the master version 2e63b33

Android version

10, 14

Device used

Redmi 9T, Pixel 7a

Problem description

Changing language/theme during form entry blocks match exactly and blocks opening the form. After that the user tries to open the form and sees info that "The form can’t be used” but at the same time there isn’t a progress bar in "Start new form”.

Steps to reproduce the problem

  1. Download user "trees” https://test.getodk.cloud/#/projects/575/app-users
  2. Go to ABC registration form.
  3. Fill the form, tap the three dots to go to settings and change the theme of the app (to create a savepoint).
  4. Wait for match exactly to run.
  5. Try to open the form.

Expected behavior

Both form entry and updates should be possible after changing language/theme during form entry.

@seadowg seadowg added this to the v2024.3 milestone Oct 30, 2024
@seadowg
Copy link
Member

seadowg commented Oct 30, 2024

This is a bug! The both form updates and entry should be possible here. It's likely that this can also be reproduced by changing languages.

@seadowg seadowg self-assigned this Oct 30, 2024
@seadowg seadowg changed the title Savepoint in an entity form blocks both match exactly and opening the form Changing theme during form entry blocks both match exactly and opening the form again Oct 30, 2024
@seadowg seadowg changed the title Changing theme during form entry blocks both match exactly and opening the form again Savepoint in an entity form blocks both match exactly and opening the form Oct 30, 2024
@seadowg
Copy link
Member

seadowg commented Oct 30, 2024

@dbemke I'm pretty sure this will only be the case for scenarios where the theme/language are changed and is actually not related to save points at all. Could you confirm that for me by playing with other save point scenarios?

@dbemke
Copy link
Author

dbemke commented Oct 30, 2024

the theme/language are changed and is actually not related to save points at all

interesting :) I'll try it out

@seadowg
Copy link
Member

seadowg commented Oct 30, 2024

@WKobus also discovered that you can cause the same problem with these steps:

  1. Finalize entity form
  2. Open it in Ready to Send
  3. Try and start the same form again

It looks like this is because we open ViewOnlyFormHierarchyActivity after loading the form and then finish FormFillingActivity without cleaning up.

@grzesiek2010
Copy link
Member

@WKobus could you file a separate issue for your case? It is a distinct one so it will be better.

@seadowg seadowg changed the title Savepoint in an entity form blocks both match exactly and opening the form Changing language/theme during form entry blocks both match exactly and opening the form Oct 30, 2024
@dbemke
Copy link
Author

dbemke commented Oct 30, 2024

I'm pretty sure this will only be the case for scenarios where the theme/language are changed and is actually not related to save points at all. Could you confirm that for me by playing with other save point scenarios?

A few tries and - savepoint after using "force stop" doesn't block match exactly or the form. I checked also if maybe it's the case with changing match exactly settings to get it running faster - it's not a factor in this issue. So as you said it's not the savepoint
causing it.

@seadowg seadowg linked a pull request Oct 30, 2024 that will close this issue
6 tasks
@seadowg
Copy link
Member

seadowg commented Oct 30, 2024

@alyblenkin it would be good to get your input on how we're thinking of fixing this. My current plan is to block access to the settings that cause the app to "restart" during form entry like this by disabling them and showing a summary like:

Screenshot 2024-10-30 at 16 23 37

And for clarity, the settings that will be disabled will be:

  • The full project management section
  • Theme
  • Language

@alyblenkin
Copy link
Collaborator

@seadowg blocking access and showing a message sounds like a good plan.

My first thought was to add some sort of pill or icon to make to more obvious if they don't read carefully, but I think that over complicates things! There is a good icon for settings alert, but not really a good warning for theme and language.

So when users go to the settings page, they’ll see the supporting text for each setting that’s not available. If they’re adjusting settings while filling out the form using the menu, should we also show the the language is disabled? I don't know if dropdown lists all support text, so we may need to show it in the dialog?

@seadowg
Copy link
Member

seadowg commented Oct 31, 2024

If they’re adjusting settings while filling out the form using the menu, should we also show the the language is disabled? I don't know if dropdown lists all support text, so we may need to show it in the dialog?

The "Setting not available..." text is used as the summary which is where the current setting would usually be shown. Just to be clear, the user is still able to change form language using the "Change language" button in the options menu - it's just the app language that they won't be able to change.

@alyblenkin
Copy link
Collaborator

Ah gotcha! That makes more sense!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: in progress
Development

Successfully merging a pull request may close this issue.

4 participants