ami/tl-vikunja
1
0
Fork 0
Code

fix(subscription): don't remove every namespace but the one subscribing to

Browse Source
This commit is contained in:
kolaente 2022-10-24 18:56:50 +02:00
parent eae555475d
commit f17bbeddec
No known key found for this signature in database
GPG Key ID: F40E70337AB24C9B
2 changed files with 11 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/components/namespace/namespace-settings-dropdown.vue
View File

@ -79,11 +79,9 @@ onMounted(() => {
function setSubscriptionInStore(sub: ISubscription) { function setSubscriptionInStore(sub: ISubscription) {
subscription.value = sub subscription.value = sub
namespaceStore.setNamespaces([ namespaceStore.setNamespaceById({
{ ...props.namespace,
...props.namespace, subscription: sub,
subscription: sub, })
},
])
} }
</script> </script>

7
src/stores/namespaces.ts
View File

@ -84,6 +84,13 @@ export const useNamespaceStore = defineStore('namespace', {
if (!namespace.lists || namespace.lists.length === 0) { if (!namespace.lists || namespace.lists.length === 0) {
namespace.lists = this.namespaces[namespaceIndex].lists namespace.lists = this.namespaces[namespaceIndex].lists
} }
// Check for each list in that namespace if it has a subscription and set it if not
namespace.lists.forEach(l => {
if (l.subscription === null || l.subscription.entity !== 'list') {
l.subscription = namespace.subscription
}
})
this.namespaces[namespaceIndex] = namespace this.namespaces[namespaceIndex] = namespace
update(namespace) update(namespace)