fix: wrong toast type on admin status change

[juliajforesti]

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

Summary by CodeRabbit

Release Notes

• Style
  • Updated notification feedback when removing admin status from a user to provide clearer success confirmation.

[dionisio-bot]

Looks like this PR is not ready to merge, because of the following issues:

• This PR is missing the 'stat: QA assured' label

Please fix the issues and try again

If you have any trouble, please check the PR guidelines

[changeset-bot]

⚠️ No Changeset found

Latest commit: 6a4d8a3

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[coderabbitai]

Walkthrough

A toast notification type is changed from "info" to "success" when removing admin status from a user, providing clearer feedback on successful admin status removal.

Changes

Cohort / File(s)	Change Summary
Admin status removal feedback apps/meteor/client/views/admin/users/hooks/useChangeAdminStatusAction.ts	Toast message type updated from "info" to "success" when removing admin status

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• chore!: remove deprecated setAdminStatus #37388: Modifies the same useChangeAdminStatusAction hook, refactoring its signature, endpoints, and messages.

Suggested labels

stat: QA assured

Suggested reviewers

• ggazzo

Poem

🐰 A toast of success, not merely info,
When admins depart, let the UI show,
Green gleams brighter than blue's gentle glow,
One small change makes feedback's true pro! ✨

Pre-merge checks and finishing touches

✅ 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 directly and accurately describes the main change: fixing the incorrect toast message type when admin status is changed, which matches the implementation detail that the toast type changed from 'info' to 'success'.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ 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/admin-status-toast-type

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

• Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 648aad5 and 6a4d8a3.

📒 Files selected for processing (1)

• apps/meteor/client/views/admin/users/hooks/useChangeAdminStatusAction.ts (1 hunks)

🔇 Additional comments (1)

apps/meteor/client/views/admin/users/hooks/useChangeAdminStatusAction.ts (1)

24-24: Good fix—improves consistency and user feedback.

Changing the toast type to 'success' aligns with line 29 (adding admin status) and provides clearer feedback that the removal action completed successfully.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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