roll-call icon indicating copy to clipboard operation
roll-call copied to clipboard

Prevent window close if recorded audio is still uploading.

Open mikeal opened this issue 8 years ago • 8 comments

If there's a recording still being uploaded we should try to prevent the user from closing the window if they try.

mikeal avatar Oct 07 '16 18:10 mikeal

How about this use case.

A person in poor network tries and audio upload is in progress. He wants to close the browser.

If above feature is implemented, the user won't be close and he will have to wait indefinitely til upload is done..

Is it a valid scenario to be consider?

imvetri avatar Oct 18 '16 12:10 imvetri

The browser won't actually let you entirely stop it from closing, the most you can do is present a "are you sure" dialog.

Google Docs handle this quite well, your changes are constantly syncing and if you haven't finished updating the backend and try to close you get a dialog letting you know that you'll lose changes. If you're in sync the dialog is never presented.

mikeal avatar Oct 18 '16 16:10 mikeal

Got it. thanks

imvetri avatar Oct 18 '16 16:10 imvetri

Hello, I'd like to work on this, if it's still open.

wizzy25 avatar Oct 22 '16 00:10 wizzy25

Yup, still open ;)

On Friday, October 21, 2016, Williams Omayuku [email protected] wrote:

Hello, I'd like to work on this, if it's still open.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/mikeal/roll-call/issues/8#issuecomment-255495235, or mute the thread https://github.com/notifications/unsubscribe-auth/AAACQ09f3siWnbRyNnT2NxBKOeMxtcQgks5q2VomgaJpZM4KRV1d .

mikeal avatar Oct 25 '16 18:10 mikeal

How can we test this? The files upload too fast

suamorales avatar Oct 29 '16 00:10 suamorales

@suamorales whatever code you end up putting in to check if the stream is completed, set it to always return false when you want to test the negative case.

mikeal avatar Oct 29 '16 02:10 mikeal

Need to add this back for the new version.

mikeal avatar Sep 19 '17 18:09 mikeal