Skip to content

Remove archaic notifications page - #2854

Closed
eric-pSAP wants to merge 2 commits into
mainfrom
eric-pSAP-remove-archaic-notifications
Closed

eric-pSAP wants to merge 2 commits into
mainfrom
eric-pSAP-remove-archaic-notifications

Conversation

@eric-pSAP

Copy link
Copy Markdown
Contributor

No description provided.

@renejeglinsky

renejeglinsky commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Hi @eric-pSAP ,
I can't quite grasp why this page should be removed. Can you give some insights?
Should we replace this page with a proper page for the notifications plugin or delete it completely?
Thanks René

@eric-pSAP

Copy link
Copy Markdown
Contributor Author

Hi @renejeglinsky,
The reason for the proposed removal is that it contradicts our notification plugin we created. The instructions are quite different from how the plugin works, which has it's own page. In addition, we could not confirm that @sap/notification-service required in the example is a real package. Removing this page would therefore make it more clear to users that they should use the notification plugin instead of following these steps.

@eric-pSAP

eric-pSAP commented Sep 17, 2026

Copy link
Copy Markdown
Contributor Author

Closed as Daniel has removed all of the archaic files in the platform/ folder. Thanks for your help!

@eric-pSAP eric-pSAP closed this Sep 17, 2026
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