Mhh, matchAll is literally only used here but I think the default is not applied because we explicitly pass null as the argument:
lib/item.js:
export const getDeleteCommand = (text = '') => { const matches = [...text.matchAll(deletePattern)] const commands = matches?.map(match => ({ number: match[1], unit: match[2] })) return commands.length ? commands[commands.length - 1] : undefined } export const hasDeleteCommand = (text) => !!getDeleteCommand(text)
api/resolvers/item.js:
if (hasDeleteCommand(old.text)) { // delete any deletion jobs that were created from a prior version of the item await clearDeletionJobs(item, models) }
reply
Thank you 🙏
reply
No worries :)
reply