Skip to content

Revert "feat(dashboard): add Bunkx shortcut in overflow menu"#17

Merged
Noelithub77 merged 1 commit into
masterfrom
revert-16-feat/dashboard-bunkx-link
Mar 31, 2026
Merged

Revert "feat(dashboard): add Bunkx shortcut in overflow menu"#17
Noelithub77 merged 1 commit into
masterfrom
revert-16-feat/dashboard-bunkx-link

Conversation

@Noelithub77
Copy link
Copy Markdown
Owner

Reverts #16
just noticed it now, ig there's no need to setup an api on the landing page and you can directly send a post request to bunkx

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 31, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ee86222c-1ddc-457c-bf88-cde5fabff034

📥 Commits

Reviewing files that changed from the base of the PR and between 997459c and 575a180.

📒 Files selected for processing (9)
  • bunkialo-landing/src/app/api/bunkx/session/[sid]/route.ts
  • bunkialo-landing/src/app/api/bunkx/session/route.ts
  • bunkialo-landing/src/app/bunkialo/page.tsx
  • bunkialo-landing/src/lib/bunkx-payload.ts
  • bunkialo-landing/src/lib/bunkx-session-store.ts
  • src/app/(tabs)/index.tsx
  • src/services/bunkx-session.ts
  • src/types/attendance.ts
  • src/utils/bunkx-payload.ts

📝 Walkthrough

Summary by CodeRabbit

  • Removed Features
    • Removed the Bunkx attendance sharing feature, including the ability to create and share attendance sessions with others.

Walkthrough

This PR removes the entire Bunkx attendance sharing feature from both the web backend and React Native mobile application, including API routes, session management, payload processing utilities, UI actions, and type definitions across 8 files.

Changes

Cohort / File(s) Summary
Web Backend API Routes
bunkialo-landing/src/app/api/bunkx/session/[sid]/route.ts, bunkialo-landing/src/app/api/bunkx/session/route.ts
Removed GET handler for session retrieval and POST handler for session creation, eliminating session-related API endpoints.
Web Backend Session & Page
bunkialo-landing/src/app/bunkialo/page.tsx, bunkialo-landing/src/lib/bunkx-session-store.ts
Removed server-side page component that consumed sessions and deleted in-memory session storage with cleanup and expiration logic.
Web Backend Types & Utilities
bunkialo-landing/src/lib/bunkx-payload.ts
Removed payload type interfaces (BunkxAttendanceRow, BunkxAttendancePayload) and runtime parser function.
Mobile App UI & Service
src/app/(tabs)/index.tsx, src/services/bunkx-session.ts
Removed "Bunkx" FAB action from dashboard and deleted HTTP service for creating remote sessions.
Mobile App Types & Utilities
src/types/attendance.ts, src/utils/bunkx-payload.ts
Removed Bunkx-related type definitions and payload encoding/transformation utilities.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested labels

codex

Poem

🐰 The Bunkx bunny hops away,
Sessions dissolve, no more play,
Files vanish, clean and neat,
A feature removed, bittersweet! 🌙

✨ 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 revert-16-feat/dashboard-bunkx-link

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.

@Noelithub77 Noelithub77 merged commit 9a7a87c into master Mar 31, 2026
3 of 4 checks passed
@Noelithub77 Noelithub77 deleted the revert-16-feat/dashboard-bunkx-link branch March 31, 2026 09:51
@coderabbitai coderabbitai Bot added the codex label Mar 31, 2026
@github-actions
Copy link
Copy Markdown

Preview

QR Code

@vishnunathasuresh
Copy link
Copy Markdown
Contributor

you r right... updated and submitted the necessary pr changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants