Skip to content

ZOOKEEPER-5061 ZooKeeper website fixes#2413

Open
yuriipalam wants to merge 9 commits into
apache:masterfrom
yuriipalam:ZOOKEEPER-5061
Open

ZOOKEEPER-5061 ZooKeeper website fixes#2413
yuriipalam wants to merge 9 commits into
apache:masterfrom
yuriipalam:ZOOKEEPER-5061

Conversation

@yuriipalam

Copy link
Copy Markdown
Contributor

Here are the website fixes to issues/bugs addressed by @phunt and @anmolnar

I also added an updated README.md file to this PR, so it's easier to review for everyone. Once the PR is approved, I will remove it and @anmolnar will update the wiki page. The file describes how the update and release flows of the website happen. I tried to write in as much detail as possible.

Thanks.

@yuriipalam

Copy link
Copy Markdown
Contributor Author

Added a few more changes.

@anmolnar anmolnar 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.

Few comments.

//

export const CURRENT_VERSION = "3.9.5";
export const CURRENT_VERSION = "3.9.10";

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.

Since we're on the master branch this should be 3.10.0

Comment on lines -166 to -175
function OlderDocsSubMenu() {
return (
<DropdownMenuSub>
<DropdownMenuSubTrigger>Older docs</DropdownMenuSubTrigger>
<DropdownMenuSubContent className="w-56 p-0">
<OlderDocsVersionList />
</DropdownMenuSubContent>
</DropdownMenuSub>
);
}

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.

Please don't remove the Older Docs section. We still need it for all the legacy docs: 3.3.x, 3.4.x, 3.5.x and everything else.

@anmolnar

Copy link
Copy Markdown
Contributor

I think the "IRC Channel" page could be replaced with ASF Slack information, because IRC is no longer maintained.

Workspace: https://the-asf.slack.com/
Channel: #zookeeper
More info: https://infra.apache.org/slack.html

@anmolnar

anmolnar commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Tests with npm run ci still fail with Nodejs v26.3.0:

Test Files  5 failed | 6 passed (11)
      Tests  39 failed | 68 passed (107)

Ignore that. You already listed the supported Node versions in the Readme. 👍

@phunt phunt 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.

lgtm - the issue of http->https is a good fix. we should do this across the codebase (separate jira/pr)


Full documentation for this release can also be found in ../../docs/index.html
Full documentation for this release can also be found at:
https://zookeeper.apache.org/doc/current/

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.

I'm not sure - but shouldn't all references be local given the doc relative to this reference is in this code base and should be used (could be updated as part of a change) rather than pointing to the release version on the website? Similar comment for all the relative -> absolute/web conversions in this commit.

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.

3 participants