Merge pull request #879 from nextcloud/queue-request

Queue contacts updates

package.json

@@ -43,6 +43,7 @@
 		"nextcloud-server": "^0.15.9",
 		"nextcloud-vue": "^0.5.1",
 		"p-limit": "^2.1.0",
+		"p-queue": "^3.0.0",
 		"uuid": "^3.3.2",
 		"v-tooltip": "^2.0.0-rc.33",
 		"vue": "^2.5.22",

src/components/ContactDetails.vue

@@ -97,7 +97,7 @@
 				<!-- using contact.key in the key and index as key to avoid conflicts between similar data and exact key -->
 				<contact-property v-for="(property, index) in sortedProperties" :key="`${index}-${contact.key}-${property.name}`" :index="index"
 					:sorted-properties="sortedProperties" :property="property" :contact="contact"
-					@updatedcontact="updateContact" />
+					@updatedcontact="debounceUpdateContact" />
 
 				<!-- addressbook change select - no last property because class is not applied here,
 					empty property because this is a required prop on regular property-select. But since
@@ -119,6 +119,7 @@
 
 <script>
 import debounce from 'debounce'
+import PQueue from 'p-queue'
 
 import rfcProps from 'Models/rfcProps'
 
@@ -128,6 +129,8 @@ import PropertySelect from './Properties/PropertySelect'
 import PropertyGroups from './Properties/PropertyGroups'
 import ContactAvatar from './ContactDetails/ContactDetailsAvatar'
 
+const updateQueue = new PQueue({ concurrency: 1 })
+
 export default {
 	name: 'ContactDetails',
 
@@ -308,7 +311,7 @@ export default {
 			},
 			set: function(data) {
 				this.contact.groups = data
-				this.updateContact()
+				this.debounceUpdateContact()
 			}
 		},
 
@@ -368,7 +371,7 @@ export default {
 		 * photo, fn, org, title
 		 */
 		debounceUpdateContact: debounce(function(e) {
-			this.updateContact()
+			updateQueue.add(this.updateContact)
 		}, 500),
 
 		// menu handling