Skip to content

Add hello demo to Portal home page#98

Closed
raidguildbot wants to merge 1 commit into
stagingfrom
codex/cr-184-add-hello-demo-to-portal-home-page
Closed

Add hello demo to Portal home page#98
raidguildbot wants to merge 1 commit into
stagingfrom
codex/cr-184-add-hello-demo-to-portal-home-page

Conversation

@raidguildbot

@raidguildbot raidguildbot commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add the exact text hello demo to the public Portal home page hero area.
  • Add an e2e assertion to the existing public-home verification helper.

Verification

  • corepack pnpm test:e2e attempted; blocked because Docker is not installed in this runtime (spawnSync docker ENOENT).
  • CI=true corepack pnpm test:e2e retried after native build approval; reached Playwright and hit the same missing Docker blocker.
  • corepack pnpm lint attempted; blocked by existing Next lint invocation/config mismatch (Invalid project directory ... /lint).
  • corepack pnpm exec eslint src/app/(frontend)/_components/PortalShell.tsx tests/e2e/app.spec.ts attempted; blocked by existing ESLint 9 migration/config mismatch.
  • corepack pnpm exec tsc --noEmit attempted; blocked by existing missing react-force-graph-2d module/type declaration.

Closes #97

Summary by CodeRabbit

  • New Features
    • Added a new text element to the public home page hero section.

@coderabbitai

coderabbitai Bot commented Jun 12, 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

Run ID: 0dfc3beb-5949-43ff-abaa-a81c95a71198

📥 Commits

Reviewing files that changed from the base of the PR and between 5cb4456 and fe1ed9b.

📒 Files selected for processing (2)
  • src/app/(frontend)/_components/PortalShell.tsx
  • tests/e2e/app.spec.ts

📝 Walkthrough

Walkthrough

A minimal home page change adds the hardcoded text "hello demo" to the public home hero section in PortalShell, with a corresponding e2e test assertion to verify the text is visible on the public home page.

Changes

Home page text addition

Layer / File(s) Summary
Add hello demo to home page
src/app/(frontend)/_components/PortalShell.tsx, tests/e2e/app.spec.ts
Hardcoded <p>hello demo</p> element added to PortalPublicHome hero section between intro copy and action buttons. E2e test asserts "hello demo" is visible on the / page.

🎯 1 (Trivial) | ⏱️ ~3 minutes


🐰 A hero needs a greeting true,
"Hello demo" now shows through,
Hardcoded text, so neat and clean,
The simplest change you've ever seen!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and specifically describes the main change: adding 'hello demo' text to the Portal home page, which matches the actual code changes made.
Linked Issues check ✅ Passed The pull request fulfills all coding requirements from issue #97: adds the exact text 'hello demo' to the home page hero area and adds corresponding e2e test assertion.
Out of Scope Changes check ✅ Passed All changes are directly related to issue #97 requirements. The modifications to PortalShell.tsx and app.spec.ts are minimal, focused, and aligned with the stated objective.

✏️ 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 codex/cr-184-add-hello-demo-to-portal-home-page

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 timed out. The project may have too many dependencies for the sandbox.


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.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a small, explicit “hello demo” text snippet to the public (unauthenticated) Portal home page hero section and updates the existing Playwright helper to assert that the text is visible, aligning with Issue #97’s request for a minimal visible content change.

Changes:

  • Render the exact text hello demo in the public home page hero area.
  • Extend the verifyPublicHome e2e helper to assert hello demo is visible.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/app/(frontend)/_components/PortalShell.tsx Adds a new hero-area paragraph that renders hello demo on the public home page.
tests/e2e/app.spec.ts Adds a Playwright visibility assertion for hello demo within the existing public-home verification helper.

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

@dekanbro dekanbro closed this Jun 12, 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.

Add hello demo to Portal home page

3 participants