binarybeach/bb-plane-fork
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

chore: Add workspace slug to should render setting link method (#6886)

Browse source 
* add workspace slug to setting link function

* add params in the function
This commit is contained in:
Saurabh Kumar 2025-04-14 17:41:47 +05:30 • committed by GitHub
parent c6953ff878
commit 24899887b2
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
4 changed files with 5 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

2
web/app/[workspaceSlug]/(projects)/settings/(with-sidebar)/mobile-header-tabs.tsx
View file

@ -20,7 +20,7 @@ export const MobileWorkspaceSettingsTabs = observer(() => {
<div className="flex-shrink-0 md:hidden sticky inset-0 flex overflow-x-auto bg-custom-background-100 z-10"> <div className="flex-shrink-0 md:hidden sticky inset-0 flex overflow-x-auto bg-custom-background-100 z-10">
{WORKSPACE_SETTINGS_LINKS.map( {WORKSPACE_SETTINGS_LINKS.map(
(item, index) => (item, index) =>
shouldRenderSettingLink(item.key) && shouldRenderSettingLink(workspaceSlug.toString(), item.key) &&
allowPermissions(item.access, EUserPermissionsLevel.WORKSPACE, workspaceSlug.toString()) && ( allowPermissions(item.access, EUserPermissionsLevel.WORKSPACE, workspaceSlug.toString()) && (
<div <div
className={`${ className={`${

2
web/app/[workspaceSlug]/(projects)/settings/(with-sidebar)/sidebar.tsx
View file

@ -28,7 +28,7 @@ export const WorkspaceSettingsSidebar = observer(() => {
<div className="flex w-full flex-col gap-1"> <div className="flex w-full flex-col gap-1">
{WORKSPACE_SETTINGS_LINKS.map( {WORKSPACE_SETTINGS_LINKS.map(
(link) => (link) =>
shouldRenderSettingLink(link.key) && shouldRenderSettingLink(workspaceSlug.toString(), link.key) &&
allowPermissions(link.access, EUserPermissionsLevel.WORKSPACE, workspaceSlug.toString()) && ( allowPermissions(link.access, EUserPermissionsLevel.WORKSPACE, workspaceSlug.toString()) && (
<Link key={link.key} href={`/${workspaceSlug}${link.href}`}> <Link key={link.key} href={`/${workspaceSlug}${link.href}`}>
<SidebarNavItem <SidebarNavItem

4
web/ce/helpers/workspace.helper.ts
View file

@ -1,2 +1,2 @@
// eslint-disable-next-line @typescript-eslint/no-unused-vars export type TRenderSettingsLink = (workspaceSlug: string, settingKey: string) => boolean;
export const shouldRenderSettingLink = (settingKey: string) => true; export const shouldRenderSettingLink: TRenderSettingsLink = (workspaceSlug, settingKey) => true;

2
web/core/components/command-palette/actions/workspace-settings-actions.tsx
View file

@ -40,7 +40,7 @@ export const CommandPaletteWorkspaceSettingsActions: React.FC<Props> = (props) =
{WORKSPACE_SETTINGS_LINKS.map( {WORKSPACE_SETTINGS_LINKS.map(
(setting) => (setting) =>
allowPermissions(setting.access, EUserPermissionsLevel.WORKSPACE, workspaceSlug.toString()) && allowPermissions(setting.access, EUserPermissionsLevel.WORKSPACE, workspaceSlug.toString()) &&
shouldRenderSettingLink(setting.key) && ( shouldRenderSettingLink(workspaceSlug.toString(), setting.key) && (
<Command.Item <Command.Item
key={setting.key} key={setting.key}
onSelect={() => redirect(`/${workspaceSlug}${setting.href}`)} onSelect={() => redirect(`/${workspaceSlug}${setting.href}`)}