Skip to content

HBASE-30127 Improve Website's SEO#8157

Merged
PDavid merged 8 commits into
apache:masterfrom
yuriipalam:HBASE-30127
Jun 4, 2026
Merged

HBASE-30127 Improve Website's SEO#8157
PDavid merged 8 commits into
apache:masterfrom
yuriipalam:HBASE-30127

Conversation

@yuriipalam
Copy link
Copy Markdown
Contributor

@yuriipalam yuriipalam commented Apr 28, 2026

There is also a small change in the hbase-site repo, please take a look:
apache/hbase-site#13

This change improves SEO (Search Engine Optimization) by adding proper cache headers, good looking 404 page, and sitemap.

404 page in dark mode:
image

Copy link
Copy Markdown
Contributor

@PDavid PDavid left a comment

Choose a reason for hiding this comment

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

Many thanks, looks really nice already. 👍

Just two small questions / comments.

Comment thread hbase-website/public/sitemap.xml Outdated
Comment thread hbase-website/package.json Outdated
@PDavid
Copy link
Copy Markdown
Contributor

PDavid commented Apr 28, 2026

It seems that the build complains about these files missing the ASF licence header:

Files with unapproved licenses:

public/robots.txt
public/sitemap.xml

Copy link
Copy Markdown
Contributor

@PDavid PDavid left a comment

Choose a reason for hiding this comment

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

Many thanks, looks good to me. 👍

@PDavid
Copy link
Copy Markdown
Contributor

PDavid commented Apr 29, 2026

Some tests failed in the PR build but they are not because of the changes in this branch.

@PDavid
Copy link
Copy Markdown
Contributor

PDavid commented May 28, 2026

Hi @yuriipalam,
Since apache/hbase-site#13 is related should we merge this PR first and then wait until the website is deployed or should we merge apache/hbase-site#13 first or the order does not matter?

@yuriipalam
Copy link
Copy Markdown
Contributor Author

Hi @PDavid
I think it's not important. We can just merge both of them at the same time. The order doesn't really matter.

@PDavid
Copy link
Copy Markdown
Contributor

PDavid commented Jun 4, 2026

Test failures in the PR build are unrelated since this is a website-only change.

@PDavid PDavid merged commit fc5b991 into apache:master Jun 4, 2026
6 of 8 checks passed
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.

2 participants