Skip to content

Keep the workbench off /develop/ - #3915

Merged
paoloredis merged 1 commit into
mainfrom
workbench-not-on-develop
Sep 7, 2026
Merged

Keep the workbench off /develop/#3915
paoloredis merged 1 commit into
mainfrom
workbench-not-on-develop

Conversation

@paoloredis

@paoloredis paoloredis commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

This change hides the workbench on the /develop/ landing page

The Redis CLI on /develop/ is a picture of one: a block of commands and
their output, there to show what Redis looks like. The workbench mounts
anyway, because the page has a terminal in it, and puts a docked console
along the bottom of a landing page that never asked for one.

A page can now say no, the way a command page says yes:
window.REDIS_WORKBENCH_NEVER, set by the template. Hugo knows which page
this is; the browser only knows a path that is prefixed differently in
every environment.

It declines outright rather than opening closed — nothing mounts, so
there is no bar, no session and no keyspace probing from this page. A
"Try it" added here later still works: RedisWorkbench.open() reports that
it cannot run and the caller opens redis.io/cli, as it did before the
dock existed.

Only /develop/ itself. Its children use the same layout and are ordinary
docs pages, examples and all.

Verified locally: no dock on /develop/ even with one left open in
localStorage, and nothing from the dock reaches the backend; the dock is
still there on /develop/data-types/hashes/ under the same layout and on
/commands/hset/.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

🧠 Redis Memory

Found 5 related items from repository history (5 new this commit):

Memory updated at eab6b84

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@paoloredis
paoloredis requested a review from a team September 3, 2026 14:05

@andy-stark-redis andy-stark-redis 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.

Works fine (and Workbench still appears on other pages where it should). Approved.

@mich-elle-luna mich-elle-luna 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.

thank you!

@paoloredis
paoloredis merged commit 1de4fe5 into main Sep 7, 2026
92 of 93 checks passed
@paoloredis
paoloredis deleted the workbench-not-on-develop branch September 7, 2026 07:59
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.

4 participants