Skip to content

fix: update readme#277

Open
yamal-alm wants to merge 1 commit into
masterfrom
udpate-pushes-readme
Open

fix: update readme#277
yamal-alm wants to merge 1 commit into
masterfrom
udpate-pushes-readme

Conversation

@yamal-alm

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings June 30, 2026 07:53
@yamal-alm yamal-alm requested review from claramunt and idenjoe June 30, 2026 07:54

Copilot AI 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.

Pull request overview

Updates the public API documentation for push-notification bridge methods to clarify brand availability and add the “feature not supported” error case.

Changes:

  • Documented that push notification subscribe/unsubscribe are only available in Mein O2 / Mein Blau.
  • Added 405 as a documented error case for “feature not supported in current brand” in both source JSDoc and README.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
src/push-notifications.ts Updates JSDoc to include brand availability and the 405 error case for push notification methods.
README.md Updates the documented API sections for push notifications to include brand availability and the 405 error case.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/push-notifications.ts
Comment thread src/push-notifications.ts
Comment thread README.md
Comment thread README.md
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.

3 participants