Skip to content

fix(status-bar)!: remove notification related extensions - #2442

Merged
eric-horodyski merged 3 commits into
mainfrom
RMET-4558
Nov 14, 2025
Merged

fix(status-bar)!: remove notification related extensions#2442
eric-horodyski merged 3 commits into
mainfrom
RMET-4558

Conversation

@jcesarmobile

Copy link
Copy Markdown
Member

Marking it as breaking in case other plugins were listening for those notifications, but the notifications are now provided by@capacitor/ios

@markemer markemer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yeah, the proper place to do this is in Capacitor iOS, and I worry about double notifications if we leave this in.

@jcesarmobile

Copy link
Copy Markdown
Member Author

There won't be double notification, these extensions on status-bar plugin were preventing the equivalent CAPBridgeViewController methods from being called, so only one fires, but not the correct one.

@eric-horodyski
eric-horodyski merged commit 8068706 into main Nov 14, 2025
3 checks passed
@eric-horodyski
eric-horodyski deleted the RMET-4558 branch November 14, 2025 17:03
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.

5 participants