Browse Source

Merge pull request #44381 from nextcloud/fix/workflows-engine-groups

fix(workflowengine): Use correct event names to make search for groups work again
Andy Scherzinger 2 months ago
parent
commit
a7dfec070a

+ 15 - 6
apps/workflowengine/src/components/Checks/RequestUserGroup.vue

@@ -22,21 +22,26 @@
 
 <template>
 	<div>
-		<NcSelect :value="currentValue"
+		<NcSelect :aria-label-combobox="t('workflowengine', 'Select groups')"
+			:aria-label-listbox="t('workflowengine', 'Groups')"
+			:clearable="false"
 			:loading="status.isLoading && groups.length === 0"
+			:placeholder="t('workflowengine', 'Type to search for group …')"
 			:options="groups"
-			:clearable="false"
+			:value="currentValue"
 			label="displayname"
-			@search-change="searchAsync"
+			@search="searchAsync"
 			@input="(value) => $emit('input', value.id)" />
 	</div>
 </template>
 
 <script>
-import NcSelect from '@nextcloud/vue/dist/Components/NcSelect.js'
-import axios from '@nextcloud/axios'
+import { translate as t } from '@nextcloud/l10n'
 import { generateOcsUrl } from '@nextcloud/router'
 
+import axios from '@nextcloud/axios'
+import NcSelect from '@nextcloud/vue/dist/Components/NcSelect.js'
+
 const groups = []
 const status = {
 	isLoading: false,
@@ -69,14 +74,18 @@ export default {
 		},
 	},
 	async mounted() {
+		// If empty, load first chunk of groups
 		if (this.groups.length === 0) {
 			await this.searchAsync('')
 		}
-		if (this.currentValue === null) {
+		// If a current group is set but not in our list of groups then search for that group
+		if (this.currentValue === null && this.value) {
 			await this.searchAsync(this.value)
 		}
 	},
 	methods: {
+		t,
+
 		searchAsync(searchQuery) {
 			if (this.status.isLoading) {
 				return

File diff suppressed because it is too large
+ 0 - 0
dist/workflowengine-workflowengine.js


File diff suppressed because it is too large
+ 0 - 0
dist/workflowengine-workflowengine.js.map


Some files were not shown because too many files changed in this diff