ami/tl-vikunja
1
0
Fork 0
Code

feat(quick add magic): allow fuzzy matching of assignees when the api results are unambigous

Browse Source
This commit is contained in:
kolaente 2023-04-04 11:05:51 +02:00 committed by konrad
parent 7b6a13dd52
commit 218a19d907
1 changed files with 25 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
src/stores/tasks.ts
View File

@ -30,6 +30,10 @@ import {useKanbanStore} from '@/stores/kanban'
import {useBaseStore} from '@/stores/base' import {useBaseStore} from '@/stores/base'
import ProjectUserService from '@/services/projectUsers' import ProjectUserService from '@/services/projectUsers'
interface MatchedAssignees extends IUser {
match: string,
}
// IDEA: maybe use a small fuzzy search here to prevent errors // IDEA: maybe use a small fuzzy search here to prevent errors
function findPropertyByValue(object, key, value) { function findPropertyByValue(object, key, value) {
return Object.values(object).find( return Object.values(object).find(
@ -37,11 +41,23 @@ function findPropertyByValue(object, key, value) {
) )
} }
function findPropertyByValueFuzzy(object, key, value) {
return Object.values(object).find(
(l) => l[key]?.toLowerCase().includes(value.toLowerCase()),
)
}
// Check if the user exists in the search results // Check if the user exists in the search results
function validateUser( function validateUser(
users: IUser[], users: IUser[],
query: IUser['username'] | IUser['name'] | IUser['email'], query: IUser['username'] | IUser['name'] | IUser['email'],
) { ) {
if (users.length === 1) {
return findPropertyByValueFuzzy(users, 'username', query) ||
findPropertyByValueFuzzy(users, 'name', query) ||
findPropertyByValueFuzzy(users, 'email', query)
}
return findPropertyByValue(users, 'username', query) || return findPropertyByValue(users, 'username', query) ||
findPropertyByValue(users, 'name', query) || findPropertyByValue(users, 'name', query) ||
findPropertyByValue(users, 'email', query) findPropertyByValue(users, 'email', query)
@ -63,14 +79,18 @@ async function addLabelToTask(task: ITask, label: ILabel) {
return response return response
} }
async function findAssignees(parsedTaskAssignees: string[], projectId: number): Promise<IUser[]> { async function findAssignees(parsedTaskAssignees: string[], projectId: number): Promise<MatchedAssignees[]> {
if (parsedTaskAssignees.length <= 0) { if (parsedTaskAssignees.length <= 0) {
return [] return []
} }
const userService = new ProjectUserService() const userService = new ProjectUserService()
const assignees = parsedTaskAssignees.map(async a => { const assignees = parsedTaskAssignees.map(async a => {
const users = await userService.getAll({projectId}, {s: a}) const users = (await userService.getAll({projectId}, {s: a}))
.map(u => ({
...u,
match: a,
}))
return validateUser(users, a) return validateUser(users, a)
}) })
@ -388,17 +408,15 @@ export const useTaskStore = defineStore('task', () => {
cancel() cancel()
throw new Error('NO_PROJECT') throw new Error('NO_PROJECT')
} }
const assignees = await findAssignees(parsedTask.assignees, foundProjectId) const assignees = await findAssignees(parsedTask.assignees, foundProjectId)
// Only clean up those assignees from the task title which actually exist // Only clean up those assignees from the task title which actually exist
let cleanedTitle = parsedTask.text let cleanedTitle = parsedTask.text
if (assignees.length > 0) { if (assignees.length > 0) {
const assigneePrefix = PREFIXES[quickAddMagicMode]?.assignee const assigneePrefix = PREFIXES[quickAddMagicMode]?.assignee
if (assigneePrefix) { if (assigneePrefix) {
cleanedTitle = cleanupItemText(cleanedTitle, assignees.map(a => a.email), assigneePrefix) cleanedTitle = cleanupItemText(cleanedTitle, assignees.map(a => a.match), assigneePrefix)
cleanedTitle = cleanupItemText(cleanedTitle, assignees.map(a => a.username), assigneePrefix)
cleanedTitle = cleanupItemText(cleanedTitle, assignees.map(a => a.name), assigneePrefix)
} }
} }