Skip to content
This repository was archived by the owner on Mar 12, 2025. It is now read-only.
This repository was archived by the owner on Mar 12, 2025. It is now read-only.

Chore: remove no longer used logic for user insight pages in tools URL #3033

Open
@nickytonline

Description

@nickytonline

We no longer need to handle cases like this, pages/${username}/${pageId}/${tool.name.toLowerCase() when building the pageNavLink in https://github.com/open-sauced/app/blob/beta/components/organisms/ToolList/nav.tsx

More context:

Nick: @brandonroberts tagging you as I see you were in this file recently. This fixes the issue by adding /explore/topic/, but this link variable is doing way too much. What is the scenario this is used for when it's pages/${username}/${pageId}/${tool.name.toLowerCase()?

I'll probably need to add a check to only add /explore/topic/ if there is no username.

Brandon: pages/${username}/${pageId}/${tool.name.toLowerCase() was for the old insight page URLs that we don't need anymore as everything is workspaces except for the explore page

For example - pages/brandonroberts/324/dashboard

The tool setup does need a rework though. We broke up those pages for workspaces, but need to go back and do the same for the explore pages.

Originally posted by @brandonroberts in #3028 (comment)

Metadata

Metadata

Assignees

No one assigned

    Labels

    chorelabel to mark dependency and documentation updatescore team workWork that the OpenSauced core team takes on

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions