1
0
Fork 0
mirror of synced 2024-07-28 17:46:09 +12:00

Fix regression when considering empty app navigation roles

This commit is contained in:
Andrew Kingston 2023-10-02 08:26:47 +01:00
parent 986069d7d7
commit bdd70d8bdb

View file

@ -3,6 +3,7 @@
import { writable } from "svelte/store" import { writable } from "svelte/store"
import { Heading, Icon, clickOutside } from "@budibase/bbui" import { Heading, Icon, clickOutside } from "@budibase/bbui"
import { FieldTypes } from "constants" import { FieldTypes } from "constants"
import { Constants } from "@budibase/frontend-core"
import active from "svelte-spa-router/active" import active from "svelte-spa-router/active"
const sdk = getContext("sdk") const sdk = getContext("sdk")
@ -103,7 +104,8 @@
let validLinks = (allLinks || []).filter(link => link.text && link.url) let validLinks = (allLinks || []).filter(link => link.text && link.url)
// Filter to only links allowed by the current role // Filter to only links allowed by the current role
return validLinks.filter(link => { return validLinks.filter(link => {
return userRoleHierarchy?.find(roleId => roleId === link.roleId) const role = link.roleId || Constants.Roles.BASIC
return userRoleHierarchy?.find(roleId => roleId === role)
}) })
} }