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

[Data collection] Changes to data on current screen not saved when app is minimized and "next" not yet clicked #2623

Closed
gino-m opened this issue Aug 9, 2024 · 11 comments
Assignees
Labels
type: bug Something isn't working
Milestone

Comments

@gino-m
Copy link
Collaborator

gino-m commented Aug 9, 2024

Please also check text and other types.

@gino-m gino-m added this to Ground Aug 9, 2024
@gino-m gino-m converted this from a draft issue Aug 9, 2024
@gino-m gino-m added the type: bug Something isn't working label Aug 9, 2024
@gino-m gino-m added this to the GA release milestone Aug 9, 2024
@anandwana001
Copy link
Collaborator

Supporting Video to this Issue

Screen.Recording.2024-08-22.at.11.56.01.AM.mov

@anandwana001
Copy link
Collaborator

So the correct bug we are seeing here is after resuming the application, the location/pointer moves back to the default place, not the place where we last left it.

Screen.Recording.2024-08-22.at.12.04.30.PM.mov

@anandwana001
Copy link
Collaborator

Screen.Recording.2024-08-23.at.8.56.33.PM.mov

Without Completing Polygon

@anandwana001
Copy link
Collaborator

@gino-m Tested all types of collection, and all are working fine. Check the attached video. I think we can close this issue.
I think I found is with the other option text area, I will check and file other issue if anything required.

Screen.Recording.2024-08-23.at.9.36.37.PM.mov

@anandwana001
Copy link
Collaborator

@shobhitagarwal1612 can we close this?

@shobhitagarwal1612
Copy link
Member

Discussed offline, the issue is when we kill the app and reopen it.

@anandwana001
Copy link
Collaborator

should we close this and create a separate issue for app kill?

@shobhitagarwal1612
Copy link
Member

@gino-m Are you still able to reproduce the issue?

@gino-m
Copy link
Collaborator Author

gino-m commented Sep 9, 2024

Based on @anandwana001 's video I think we can close this one. What is the issue # of the "app kill" case? We should be sure to enumerate the remaining "draft" states to be saved in the issue.

@anandwana001
Copy link
Collaborator

anandwana001 commented Sep 9, 2024

Based on @anandwana001 's video I think we can close this one. What is the issue # of the "app kill" case? We should be sure to enumerate the remaining "draft" states to be saved in the issue.

raised PR for kill case, which includes partial load, so kill case and partial load for all type of task will get done in one PR.
#2720

@jcqli jcqli moved this from Todo to In Progress in Ground Sep 11, 2024
@anandwana001
Copy link
Collaborator

Based on @anandwana001 's video I think we can close this one. What is the issue # of the "app kill" case? We should be sure to enumerate the remaining "draft" states to be saved in the issue.

Closing this issue.

@github-project-automation github-project-automation bot moved this from In Progress to Done in Ground Sep 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working
Projects
Status: Done
Development

No branches or pull requests

3 participants