Skip to content

feat: delete user front end#8877

Open
csiyang wants to merge 15 commits intosiyangcao/nes-1454-deleteuser-backendfrom
siyangcao/nes-1455-deleteuser-frontend
Open

feat: delete user front end#8877
csiyang wants to merge 15 commits intosiyangcao/nes-1454-deleteuser-backendfrom
siyangcao/nes-1455-deleteuser-frontend

Conversation

@csiyang
Copy link
Copy Markdown
Contributor

@csiyang csiyang commented Mar 19, 2026

No description provided.

@csiyang @claude
feat: user delete frontend (NES-1455)
c75e7e6 
Add user deletion UI for superAdmin users:
- UserDelete component with check/confirm flow
- Real-time log streaming via subscription
- Auto-scrolling log display
- Navigation link in superAdmin menu
- Confirmation dialog before deletion
- Test coverage for component rendering and auth

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
@csiyang csiyang self-assigned this Mar 19, 2026
@linear
Copy link
Copy Markdown

linear bot commented Mar 19, 2026

NES-1455 deleteUser frontend

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 19, 2026 •
edited
Loading

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 37719905-ae0a-4441-a5d8-8d940a653cea

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch siyangcao/nes-1455-deleteuser-frontend

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.

@csiyang csiyang changed the title feat: delete user backend feat: delete user front end Mar 19, 2026
@csiyang csiyang changed the base branch from main to siyangcao/nes-1454-deleteuser-backend March 19, 2026 01:53
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 19, 2026 •
edited
Loading

Warnings
⚠️ ❗ Big PR (827 changes)

(change count - 827): Pull Request size seems relatively large. If Pull Request contains multiple changes, split each into separate PR will helps faster, easier review.

Generated by 🚫 dangerJS against e74c262

@csiyang csiyang marked this pull request as ready for review March 25, 2026 22:13
@csiyang
Copy link
Copy Markdown
Contributor Author

csiyang commented Mar 25, 2026

@coderabbitai review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 25, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@csiyang
Copy link
Copy Markdown
Contributor Author

csiyang commented Mar 26, 2026

@coderabbitai review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 26, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@stage-branch-merger
Copy link
Copy Markdown

stage-branch-merger bot commented Mar 30, 2026

I see you added the "on stage" label, I'll get this merged to the stage branch!

@stage-branch-merger
Copy link
Copy Markdown

stage-branch-merger bot commented Mar 30, 2026

Merge conflict attempting to merge this into stage. Please fix manually.

@csiyang csiyang requested a review from mikeallisonJS March 30, 2026 19:12
@csiyang csiyang mentioned this pull request Mar 30, 2026
Closed
stage-branch-merger bot added a commit that referenced this pull request Mar 30, 2026
@stage-branch-merger
Merge branch 'siyangcao/nes-1455-deleteuser-frontend' (PR #8877) into…
d853e50 
… stage
@stage-branch-merger
Copy link
Copy Markdown

stage-branch-merger bot commented Mar 30, 2026

Merge conflict attempting to merge this into stage. Please fix manually.

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

Reviewers

@mikeallisonJS mikeallisonJS Awaiting requested review from mikeallisonJS

Assignees

@csiyang csiyang

Labels

on stage

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@csiyang