Skip to content

fix: adjust export for useSnackbarMessage - #223

Merged
tomrndom merged 2 commits into
mainfrom
fix/export-use-snackbar-message
Apr 21, 2026
Merged

fix: adjust export for useSnackbarMessage#223
tomrndom merged 2 commits into
mainfrom
fix/export-use-snackbar-message

Conversation

@tomrndom

@tomrndom tomrndom commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

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

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

Summary by CodeRabbit

  • Refactor

    • Streamlined notification hook imports for improved developer experience.
  • Chores

    • Removed unused build configuration entries.

Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
@tomrndom
tomrndom requested a review from santipalenque April 21, 2026 20:44
@coderabbitai

coderabbitai Bot commented Apr 21, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 628e232c-7db8-4e67-aacd-7a60b156351f

📥 Commits

Reviewing files that changed from the base of the PR and between 433c353 and 22d7e73.

📒 Files selected for processing (2)
  • src/components/mui/SnackbarNotification/index.js
  • webpack.common.js
💤 Files with no reviewable changes (1)
  • webpack.common.js

📝 Walkthrough

Walkthrough

The PR re-exports the useSnackbarMessage hook from the SnackbarNotification module's index file, allowing direct imports from the module. Concurrently, the dedicated webpack entry for the Context module is removed.

Changes

Cohort / File(s) Summary
Hook Re-export
src/components/mui/SnackbarNotification/index.js
Added named re-export of useSnackbarMessage from ./Context, enabling direct imports from the module's index.
Webpack Configuration
webpack.common.js
Removed webpack entry components/mui/snackbar-notification-context that previously mapped to the Context module.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • smarcet

Poem

🐰 A hook hops home to index lane,
No Context detours need maintain,
Webpack grows lighter, cleaner, lean—
The simplest change we've ever seen! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 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 changes: adding a re-export for useSnackbarMessage hook and removing the webpack entry point for the context module.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

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

✨ 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 fix/export-use-snackbar-message

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.

@santipalenque santipalenque left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tomrndom
tomrndom merged commit 841c37f into main Apr 21, 2026
5 checks passed
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