Skip to content

This Removed the hide explore page environmental variable.#1710

Open
brucefrankstad26-beep wants to merge 3 commits into
mainfrom
1677-app-remove-hide-explore-page-environmental-variable
Open

This Removed the hide explore page environmental variable.#1710
brucefrankstad26-beep wants to merge 3 commits into
mainfrom
1677-app-remove-hide-explore-page-environmental-variable

Conversation

@brucefrankstad26-beep

Copy link
Copy Markdown
Collaborator

No description provided.

@brucefrankstad26-beep brucefrankstad26-beep linked an issue Jun 17, 2026 that may be closed by this pull request
Comment thread app/src/components/navigation/navigationItems.ts Outdated

@ivanslabbert ivanslabbert left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also check the env.example file and ensure the env var is not there anymore

@@ -39,7 +39,6 @@ export function getNavigationItems(t: (key: string) => string) {
];

// Exclude the "Explore" navigation item if the env variable "VITE_HIDE_EXPLORE" is set to "true"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also fix comment to not refer to the env var anymore

@@ -39,7 +39,5 @@ export function getNavigationItems(t: (key: string) => string) {
];

// Exclude the "Explore" navigation item if the env variable "VITE_HIDE_EXPLORE" is set to "true"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should still be removed

Suggested change
// Exclude the "Explore" navigation item if the env variable "VITE_HIDE_EXPLORE" is set to "true"

@MrDirkelz MrDirkelz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also this branch should be rebased perhaps.

@brucefrankstad26-beep brucefrankstad26-beep force-pushed the 1677-app-remove-hide-explore-page-environmental-variable branch from 3ad7ec3 to aef2a8c Compare June 25, 2026 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

APP: Remove hide explore page environmental variable.

3 participants