Skip to content

[add] blog 1706#405

Merged
apsinghdev merged 1 commit into
mainfrom
blog/1706
Jun 17, 2026
Merged

[add] blog 1706#405
apsinghdev merged 1 commit into
mainfrom
blog/1706

Conversation

@apsinghdev

@apsinghdev apsinghdev commented Jun 17, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

Release Notes

  • New Features
    • Added a new blog post featuring author background, project highlights, and current updates
    • Updated the hero section link to direct to the new blog post

@vercel

vercel Bot commented Jun 17, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
opensox-website Ready Ready Preview, Comment Jun 17, 2026 11:08am

@coderabbitai

coderabbitai Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

A new MDX blog post (my-work-so-far.mdx) is added with frontmatter and body content covering the author's projects, metrics, and credentials. The Hero section's anchor is updated from an external GitHub Gist URL to the new internal route /blog/my-work-so-far.

Changes

Blog Post Addition and Hero Link Update

Layer / File(s) Summary
New blog post and Hero link
apps/web/src/content/blog/my-work-so-far.mdx, apps/web/src/components/landing-sections/Hero.tsx
Adds the full MDX post with YAML frontmatter (title, date, description, author, tag) and structured body; updates the Hero anchor destination from a GitHub Gist URL to the internal route /blog/my-work-so-far.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • apsinghdev/opensox#147: Adds the blog listing/navigation infrastructure (/blogs page, header/footer links) that this PR's new post and internal route depend on.
  • apsinghdev/opensox#398: Also modifies Hero.tsx's blog-related hyperlink, directly touching the same line changed in this PR.

Poem

🐇 A gist link once roamed in the wild,
Now tamed to a path, neat and styled.
A blog post is born, words take their place,
Projects and metrics shown with grace.
Hop along, reader — the story's compiled! 🌟

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title '[add] blog 1706' is vague and generic, using a non-descriptive placeholder number without clarifying what blog content is being added. Replace with a descriptive title that summarizes the blog content, such as '[add] blog post about work and projects' or '[add] my-work-so-far blog post'.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch blog/1706

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot 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.

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
apps/web/src/components/landing-sections/Hero.tsx (1)

99-107: ⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Remove target="_blank" and rel from internal link.

The link now points to an internal route (/blog/my-work-so-far), but retains target="_blank" and rel="noopener noreferrer" attributes intended for external links. Opening internal links in a new tab degrades the user experience and is not idiomatic Next.js practice.

✏️ Proposed fix
         <Link
-            href="/blog/my-work-so-far"
-            target="_blank"
-            rel="noopener noreferrer"
+            href="/blog/my-work-so-far"
             className="inline-flex items-center gap-1 border-b border-text-secondary/60 hover:border-text-primary hover:text-text-primary cursor-pointer transition-colors duration-300"
           >
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@apps/web/src/components/landing-sections/Hero.tsx` around lines 99 - 107, The
Link component in the Hero section pointing to the internal route
`/blog/my-work-so-far` currently has `target="_blank"` and `rel="noopener
noreferrer"` attributes that are intended only for external links. Remove both
the `target="_blank"` attribute and the `rel="noopener noreferrer"` attribute
from this Link component to follow Next.js idiomatic practices and ensure
internal navigation opens in the current tab, providing a better user
experience.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@apps/web/src/content/blog/my-work-so-far.mdx`:
- Line 9: The blog post my-work-so-far.mdx contains multiple capitalization,
grammar, and spelling errors throughout the content. Fix the following issues:
change "i'm" to "I'm", "twitter" to "Twitter", "Real time" to "Real-time",
"transcation" to "transaction", "youtube" to "YouTube", "on the basis of" to
either "due to" or "from", "Github" to "GitHub", and at the beginning of a
sentence change "note:" to "Note:" and "september" to "September". These
corrections ensure proper capitalization of proper nouns, correct spelling,
appropriate hyphenation for compound adjectives, and proper sentence
capitalization throughout the blog post.

---

Outside diff comments:
In `@apps/web/src/components/landing-sections/Hero.tsx`:
- Around line 99-107: The Link component in the Hero section pointing to the
internal route `/blog/my-work-so-far` currently has `target="_blank"` and
`rel="noopener noreferrer"` attributes that are intended only for external
links. Remove both the `target="_blank"` attribute and the `rel="noopener
noreferrer"` attribute from this Link component to follow Next.js idiomatic
practices and ensure internal navigation opens in the current tab, providing a
better user experience.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 09189679-b682-4eaf-bbfb-15b35178b847

📥 Commits

Reviewing files that changed from the base of the PR and between da5e3c3 and a9a3270.

📒 Files selected for processing (2)
  • apps/web/src/components/landing-sections/Hero.tsx
  • apps/web/src/content/blog/my-work-so-far.mdx

Comment thread apps/web/src/content/blog/my-work-so-far.mdx
@apsinghdev
apsinghdev merged commit bd31d94 into main Jun 17, 2026
3 checks passed
@apsinghdev
apsinghdev deleted the blog/1706 branch June 17, 2026 11:10
@coderabbitai coderabbitai Bot mentioned this pull request Jul 5, 2026
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.

1 participant