Skip to content

refactor: centralize notification logic with NotificationProvider#942

Merged
enrimon15 merged 3 commits intomainfrom
refactor/notification-provider-common-component
Dec 15, 2025
Merged

refactor: centralize notification logic with NotificationProvider#942
enrimon15 merged 3 commits intomainfrom
refactor/notification-provider-common-component

Conversation

@enrimon15
Copy link
Contributor

What type of PR is this?

  • 🔨 Refactor
  • ✨ Feature
  • ⛑️ Bug Fix
  • 🚀 Optimization
  • 📄 Documentation Update
  • ⚠️ Breaking change ℹ️

List of Changes

  • centralize notification logic and replace local notify state with context

Related Task

  • OI-451

Does this PR need a Changeset?

A changeset is a description of changes that should be included in the changelog and release notes.
If your PR includes significant changes that must be versioned, run yarn changeset from the root of the repo and select oneidentity plus any other affected package in order to create a changeset.

  • 🦋 I have added a changeset by running yarn changeset
  • ❌ This PR doesn't require a changeset (e.g., documentation, config changes)

How Has This Been Tested?

  • Local tests

Screenshots (if appropriate)

@enrimon15 enrimon15 requested a review from a team as a code owner December 15, 2025 11:20
@changeset-bot
Copy link

changeset-bot bot commented Dec 15, 2025

🦋 Changeset detected

Latest commit: 8e6fc3f

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

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

@enrimon15 enrimon15 merged commit 1c3268e into main Dec 15, 2025
11 of 15 checks passed
@enrimon15 enrimon15 deleted the refactor/notification-provider-common-component branch December 15, 2025 17:10
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