Skip to content

Khix/hacker guide - #561

Merged
DVidal1205 merged 2 commits into
mainfrom
khix/hacker-guide
Sep 12, 2026
Merged

Khix/hacker guide#561
DVidal1205 merged 2 commits into
mainfrom
khix/hacker-guide

Conversation

@morallyearlgrey

Copy link
Copy Markdown
Contributor

Why
Hackers need a clear and accessible way to open the Knight Hacks IX Hacker’s Guide from the portal.
What

  • Unlocked the Hacker’s Guide dashboard card and tool dock shortcut.
  • Linked both entry points to the centralized portal configuration.
  • Updated the guide URL to the public Notion page.
  • Configured the guide to open safely in a new tab.
    Test Plan
  • Verified both Hacker’s Guide entry points open the correct public Notion page.
  • Tested the dashboard on desktop and mobile layouts.
  • Confirmed all 14 KHIX tests pass.
  • Confirmed formatting, lint, and TypeScript checks pass.
    Checklist
  • Database: No schema changes, OR I ran pnpm db:generate and committed the generated files in packages/db/drizzle/
  • Environment Variables: No environment variables changed, OR I have contacted the Development Lead to modify them on Coolify BEFORE merging.

@coderabbitai

coderabbitai Bot commented Sep 11, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: QUIET

Plan: Advanced

Run ID: 10a1d879-c202-480e-acb6-df5f1e163e74

📥 Commits

Reviewing files that changed from the base of the PR and between 3cd0aec and 0d53967.

📒 Files selected for processing (2)
  • apps/2026/src/app/(portal)/_components/khix-dashboard.tsx
  • apps/2026/src/lib/portal-config.ts

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The portal configuration now uses the new hackers-guide URL. The dashboard passes this URL to StatusAction and ToolDock. The disabled guide button and locked guide tile now open the configured external guide.

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 0d539

The configured URL is consistently wired to both guide entry points, with no actionable merge-blocking risk established.

🚥 Pre-merge checks | ✅ 6 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title describes the Hacker’s Guide change but does not start with an issue number in brackets and does not meet the required format. Replace the title with a format such as "[#123] Unlock Hacker’s Guide access" using the correct issue number and keeping it under 72 characters.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 4 functions across 2 files. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (6 passed)
Check name Status Explanation
Description check ✅ Passed The description explains the Hacker’s Guide changes, configuration update, testing, and checklist status.
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.
No Hardcoded Secrets ✅ Passed No hardcoded credential was introduced. The PR changes only dashboard wiring and a public Notion URL. Scans of all added lines and both changed files found no API key, password, token, secret value, b…
Validated Env Access ✅ Passed The pull request changes only khix-dashboard.tsx and portal-config.ts. Neither file contains process.env, and no added line contains direct environment access. The only raw process.env referen…
No Typescript Escape Hatches ✅ Passed The pull request adds no any type, @ts-ignore, @ts-expect-error, or !. non-null assertion. The only any match is unchanged prose, not a type.
  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch khix/hacker-guide

Warning

Some tools did not complete. Review the errors below.

🔧 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.

apps/2026/src/app/(portal)/_components/khix-dashboard.tsx

ESLint skipped: missing config or dependency (missing-dependency). The ESLint configuration references a package that is not available in the sandbox.

apps/2026/src/lib/portal-config.ts

ESLint skipped: the matched ESLint configuration already failed (missing-dependency).


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.

@DVidal1205
DVidal1205 added this pull request to the merge queue Sep 12, 2026
Merged via the queue into main with commit 1f1f6ab Sep 12, 2026
13 checks passed
@DVidal1205
DVidal1205 deleted the khix/hacker-guide branch September 12, 2026 01:42
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