Skip to content

fix: trigger sponsor users table update after process acess request#780

Open
tomrndom wants to merge 1 commit intomasterfrom
fix/sponsor-user-list-process-update
Open

fix: trigger sponsor users table update after process acess request#780
tomrndom wants to merge 1 commit intomasterfrom
fix/sponsor-user-list-process-update

Conversation

@tomrndom
Copy link

@tomrndom tomrndom commented Feb 4, 2026

ref: https://app.clickup.com/t/86b81euxr

Signed-off-by: Tomás Castillo tcastilloboireau@gmail.com

Summary by CodeRabbit

  • Bug Fixes
    • Sponsor users list now properly refreshes when approving sponsor user requests, ensuring data remains synchronized across the interface.

Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
@tomrndom tomrndom requested a review from smarcet February 4, 2026 21:13
@coderabbitai
Copy link

coderabbitai bot commented Feb 4, 2026

📝 Walkthrough

Walkthrough

A single dispatch call was added to the processSponsorUserRequest function to refresh the sponsor users list after successfully approving a sponsor user request. The change ensures both the sponsor users and sponsor user requests lists are updated.

Changes

Cohort / File(s) Summary
Sponsor User Request Approval
src/actions/sponsor-users-actions.js
Added getSponsorUsers() dispatch call after successful approval to refresh the sponsor users list alongside the sponsor user requests list.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • smarcet

Poem

🐰 A sponsor list now gleams so bright,
With one dispatch, all data's right,
Approve, refresh, and all is done—
Two lists in sync, our work is fun! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: triggering a sponsor users table update after processing an access request, which matches the code modification of dispatching getSponsorUsers() after approval.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/sponsor-user-list-process-update

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant