Skip to content

chore: remove AFX hiring ad from homepage - #129

Merged
elrrrrrrr merged 1 commit into
masterfrom
agent/remove-home-afx-hiring-ad
Jul 24, 2026
Merged

chore: remove AFX hiring ad from homepage#129
elrrrrrrr merged 1 commit into
masterfrom
agent/remove-home-afx-hiring-ad

Conversation

@elrrrrrrr

@elrrrrrrr elrrrrrrr commented Jul 24, 2026

Copy link
Copy Markdown
Member

What changed

  • Removed the AFX hiring ad component from the homepage.
  • Left the shared ad component and package detail page unchanged.
  • Added an inline comment documenting why the homepage slot is disabled.

Why

The AFX Experience Technology Department hiring campaign has ended, so the homepage recruitment banner should no longer be displayed.

Impact

Homepage visitors will no longer see the AFX recruitment banner. Other pages are unaffected.

Validation

  • git diff --check passed.
  • Lint could not run because project dependencies are not installed; a dependency install attempt produced no output and was stopped without generating additional tracked changes.

Summary by CodeRabbit

  • Content Updates
    • Removed the recruitment advertisement from the page.
    • No other page content or layout was changed.

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@coderabbitai

coderabbitai Bot commented Jul 24, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: ff8b8fff-dd8f-45d4-93f8-1ec52a0016fd

📥 Commits

Reviewing files that changed from the base of the PR and between 21d0b09 and a2dd717.

📒 Files selected for processing (1)
  • src/pages/index.tsx

📝 Walkthrough

Walkthrough

The homepage removes the AdHire import and replaces its rendered component with a JSX comment stating that the AFX recruitment advertisement is offline.

Changes

Homepage recruitment advertisement

Layer / File(s) Summary
Disable recruitment advertisement
src/pages/index.tsx
Replaces the rendered AdHire component with an offline JSX comment.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

  • cnpm/cnpmweb#109: Adds and renders AdHire in the same homepage component, directly conflicting with this removal.

Suggested reviewers: killagu

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main homepage change and is concise.
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.
✨ 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 agent/remove-home-afx-hiring-ad

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint install failed: private package registry requires authentication. Disable ESLint in CodeRabbit settings or use public packages.


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.

@vercel

vercel Bot commented Jul 24, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
cnpmweb Ready Ready Preview, Comment Jul 24, 2026 11:14am

@elrrrrrrr
elrrrrrrr requested review from fengmk2 and killagu July 24, 2026 11:06
@elrrrrrrr
elrrrrrrr marked this pull request as ready for review July 24, 2026 11:06
Copilot AI review requested due to automatic review settings July 24, 2026 11:06
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

Copilot AI 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.

Pull request overview

Removes the now-expired AFX Experience Technology Department hiring banner from the homepage while documenting the intentional disablement in-place.

Changes:

  • Removed the AdHire component import and render from the homepage (src/pages/index.tsx).
  • Added an inline JSX comment noting that the AFX recruiting ad has been taken offline.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@killagu killagu 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

@elrrrrrrr
elrrrrrrr merged commit 03bc734 into master Jul 24, 2026
11 of 13 checks passed
@elrrrrrrr
elrrrrrrr deleted the agent/remove-home-afx-hiring-ad branch July 24, 2026 11:08
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