Skip to content

docs: update Orama link to their new docs domain - #9146

Merged
avivkeller merged 3 commits into
nodejs:mainfrom
ssmurfgg04-gif:fix-orama-link
Sep 9, 2026
Merged

docs: update Orama link to their new docs domain#9146
avivkeller merged 3 commits into
nodejs:mainfrom
ssmurfgg04-gif:fix-orama-link

Conversation

@ssmurfgg04-gif

Copy link
Copy Markdown
Contributor

What

Updated the Orama link in the README from docs.oramasearch.com to docs.orama.com.

Why

The old domain no longer resolves. Orama moved their docs to orama.com.

Verification

docs.oramasearch.com fails DNS resolution, docs.orama.com returns 200.

@ssmurfgg04-gif
ssmurfgg04-gif requested a review from a team as a code owner September 6, 2026 19:35
@vercel

vercel Bot commented Sep 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
nodejs-org Ready Ready Preview Sep 8, 2026 5:02pm UTC

Request Review

@bmuenzenmeyer

Copy link
Copy Markdown
Contributor

I wonder if this is the only place to consider this

@ssmurfgg04-gif
ssmurfgg04-gif requested a review from a team as a code owner September 8, 2026 12:26
@ssmurfgg04-gif

Copy link
Copy Markdown
Contributor Author

Good question. I searched the whole repo for the old domain. I found two more links in the website redesign blog post and I have updated them in this PR. The only other hit is an image host pattern for website-assets.oramasearch.com in next.config.mjs. That host no longer resolves, but it is a different service from the docs links, so I left it out of this PR. Happy to remove it too if you want.

@avivkeller

Copy link
Copy Markdown
Member

You can remove it, IMO

@ovflowd

ovflowd commented Sep 8, 2026

Copy link
Copy Markdown
Member

You can remove it, IMO

Agreed, you can remove the images, they are broken anyways...

The domain no longer resolves and no content references it anymore,
so the remote pattern is not needed.
@ssmurfgg04-gif

Copy link
Copy Markdown
Contributor Author

Done in 41a768b. I pulled the website-assets.oramasearch.com entry out of remotePatterns in apps/site/next.config.mjs. That was the last reference to the domain anywhere in the repo, so nothing points at the decommissioned host anymore. Thanks both for the quick turnaround on review.

@codecov

codecov Bot commented Sep 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.12%. Comparing base (aa05546) to head (41a768b).
⚠️ Report is 5 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #9146   +/-   ##
=======================================
  Coverage   86.12%   86.12%           
=======================================
  Files          86       86           
  Lines        6046     6046           
  Branches      357      357           
=======================================
  Hits         5207     5207           
  Misses        835      835           
  Partials        4        4           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

📦 Build Size Comparison

Summary

Metric Value
Old Total First Load JS 7.06 MB
New Total First Load JS 7.06 MB
Delta 0 B (0.00%)

@avivkeller
avivkeller enabled auto-merge September 9, 2026 19:55
@avivkeller
avivkeller disabled auto-merge September 9, 2026 19:55
@avivkeller
avivkeller merged commit 69249e0 into nodejs:main Sep 9, 2026
16 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.

6 participants