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

Saving reverts changes which happened during the save. #923

Closed
RobertZenz opened this issue Feb 7, 2019 · 13 comments
Closed

Saving reverts changes which happened during the save. #923

RobertZenz opened this issue Feb 7, 2019 · 13 comments
Labels
1. to develop Accepted and waiting to be taken care of bug Something isn't working high High priority

Comments

@RobertZenz
Copy link

When editing a contact the contact is being automatically saved when you stop typing. Any changes you make during the saving process (the spinner is visible in the right upper corner) are being reverted the moment saving is completed.

This is especially bad when you, for example, type in a phone number while reading it from a different source. You start typing, then look at the source and this pause causes the save, so there's a chance that you'll start typing during the save and any input you've done during the saving process will be reverted, potentially leaving you with a partially filled in field.

Steps to reproduce the behavior

  1. Open a contact.
  2. Start typing in an input field, stop for a few seconds and begin typing as soon as the saving starts.

When saving finishes, everything you've typed during the saving will be reverted.

Expected behavior

Changes the user put into the field should never be silently reverted.

Actual behavior

They are silently reverted when the save finishes.

Note

It for sure is my server who is not as fast as a server could be, but this hasn't been a problem until the upgrade to 15.

@RobertZenz RobertZenz changed the title Saving reverts changes which happened during the change. Saving reverts changes which happened during the save. Feb 7, 2019
@nextcloud-bot
Copy link
Member

GitMate.io thinks possibly related issues are #314 (Saved searches), #616 ([Vue] Save groups on change), #559 (Changes not saved), #562 (Changes not saved on Contacts), and #533 (Changes are not saved).

@nextcloud-bot nextcloud-bot added the bug Something isn't working label Feb 7, 2019
@skjnldsv
Copy link
Member

Hello @RobertZenz :)

Thanks for your report. I'm guessing you're on contacts 3.0.x?

@skjnldsv skjnldsv added the needs info Not enough information provided label Feb 10, 2019
@RobertZenz
Copy link
Author

Exactly, 3.0.3 to be exact.

@skjnldsv
Copy link
Member

@RobertZenz thanks.
I cannot reproduce. Every change I do are still sent to the server and kept as it is :/

As you can see (I enabled bandwidth limiter), multiple save doesn't change the edits I do to the contact.

peek 12-02-2019 14-07

@RobertZenz
Copy link
Author

I could not clearly see whether you were typing while the saving was in progress. Can I do something to debug this on my end?

@skjnldsv
Copy link
Member

@RobertZenz I'll try with an even longer bandwidth multiplier.
This happens when you use your phone, right?

@skjnldsv
Copy link
Member

@RobertZenz Okay I was able to reproduce the issue just now!

@skjnldsv skjnldsv added 1. to develop Accepted and waiting to be taken care of high High priority and removed needs info Not enough information provided labels Feb 13, 2019
@RobertZenz
Copy link
Author

Glad to hear that you could reproduce it.

Well, in my case it is not the bandwidth of the connection, it is my server directly. I'm using an oldish (10 years approaching Atom) as my main server, which is quite fast enough for the two users it needs to serve to. But from time to time it lacks a little.

@raimund-schluessler
Copy link
Member

@skjnldsv How do I enable bandwidth limiter? I would like to test it with Tasks as well.

@skjnldsv
Copy link
Member

@raimund-schluessler on the network tab of your dev tools, you can set various speed limits :)
capture d ecran_2019-02-14_14-51-03

@jaller94
Copy link

I am in Vancouver, Canada and my home server is located in Germany. The restore on a successful autosave in version 3.0.3 makes the app’s input fields unusable for me.
The effect is strongest when typing in bursts, interrupted by a brief pause for autosave to start.

2019-02-27 18 41 50

@JaviLib
Copy link

JaviLib commented Mar 2, 2019

I dont know it this is related, but now everytime I try to edit a contact, it is never saved. The circle is constantly running indicating that it is saving, but it never gets saved.

@skjnldsv
Copy link
Member

skjnldsv commented Mar 2, 2019

@JavierRSobrino anything in your browser's console?

How to access your browser console (Click to expand)

Chrome

  • Press either CTRL + SHIFT + J to open the “console” tab of the Developer Tools.
  • Alternative method:
    1. Press either CTRL + SHIFT + I or F12 to open the Developer Tools.
    2. Click the “console” tab.

Safari

  • Press CMD + ALT + I to open the Web Inspector.
  • See Chrome’s step 2. (Chrome and Safari have pretty much identical dev tools.)

IE9

  1. Press F12 to open the developer tools.
  2. Click the “console” tab.

Firefox

  • Press CTRL + SHIFT + K to open the Web console (COMMAND + SHIFT + K on Macs).
  • or, if Firebug is installed (recommended):
    1. Press F12 to open Firebug.
    2. Click on the “console” tab.

Opera

  1. Press CTRL + SHIFT + I to open Dragonfly.
  2. Click on the “console” tab.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of bug Something isn't working high High priority
Projects
None yet
Development

No branches or pull requests

6 participants