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

After using Save icon in form with background recording, recording is not saved #4397

Open
kkrawczyk123 opened this issue Feb 10, 2021 · 1 comment
Assignees

Comments

@kkrawczyk123
Copy link
Contributor

Software and hardware versions

Collect v1.29.x, Android vx.x.x, device used...

Problem description

When testing #4392 I have noticed that the behavior of save using save icon and save on dialog differs. When save on dialog recording is saved but when save using icon and then leave form ignoring further changes recording is not saved. It might be confusing.

Steps to reproduce the problem

  1. Open form with background recording
  2. Click on Save icon in form
  3. Use back button
  4. Ignore changes on displayed dialog
  5. Open files - instance is saved but recording is ignored.
    ezgif com-video-to-gif (51)

Expected behavior

The recording should not be ignored, it should be saved like after saving on save/ignore changes dialog.

Other information

Things you tried, stack traces, related issues, suggestions on how to fix it...

@lognaturel
Copy link
Member

Thanks, this is a good one. I doubt it's very common that folks are saving, making changes, and then deciding to discard just those changes but it probably does happen and losing a background recording in that case is definitely unexpected. I think we should consider treating the in-form save as the end of one form-filling session and the start of another so that background audio would be saved and possibly stitched together.

@seadowg seadowg added this to the v2022.3 milestone Apr 16, 2022
@seadowg seadowg modified the milestones: v2022.3, v2022.4 Jun 22, 2022
@grzesiek2010 grzesiek2010 self-assigned this Jul 12, 2022
@seadowg seadowg modified the milestones: v2022.4, v2022.4.1 Nov 8, 2022
@seadowg seadowg modified the milestones: v2022.4, v2022.4.1 Nov 15, 2022
@seadowg seadowg modified the milestones: v2022.4.x, v2023.1 Dec 6, 2022
@seadowg seadowg modified the milestones: v2023.1, v2023.2 Feb 24, 2023
@seadowg seadowg removed this from the v2023.2 milestone Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ready
Development

Successfully merging a pull request may close this issue.

4 participants