Skip to content

docs: add PR conventions and clean up build guide formatting#20

Merged
fadi-george merged 3 commits intomainfrom
fg/clear-all-notifications
Mar 9, 2026
Merged

docs: add PR conventions and clean up build guide formatting#20
fadi-george merged 3 commits intomainfrom
fg/clear-all-notifications

Conversation

@fadi-george
Copy link
Copy Markdown
Contributor

@fadi-george fadi-george commented Mar 9, 2026

Description

Add a Cursor rule for PR conventions, clean up build guide formatting, and add a "Clear All" notifications button to the Send Push Notification section.

Details

  • Added .cursor/rules/pr-conventions.mdc with guidelines for PR titles (conventional commit prefixes) and body structure
  • Added a CLEAR ALL button to Prompt 2.3 (Send Push Notification Section) that calls OneSignal.Notifications.clearAllNotifications() to remove all delivered notifications
  • Fixed indentation and spacing throughout demo/build.md: removed extra leading spaces on code blocks and shell commands, added blank lines before lists to comply with standard markdown formatting

@fadi-george fadi-george requested a review from a team as a code owner March 9, 2026 23:03
Comment thread demo/build.md Outdated
@fadi-george fadi-george requested a review from nan-li March 9, 2026 23:39
@fadi-george fadi-george merged commit 616dd98 into main Mar 9, 2026
6 checks passed
@fadi-george fadi-george deleted the fg/clear-all-notifications branch March 9, 2026 23:55
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