Skip to content

fix(web): remove extra colon in Pushover notification example#168

Merged
s0up4200 merged 1 commit into
developfrom
fix/pushover-typo-161
Jan 24, 2026
Merged

fix(web): remove extra colon in Pushover notification example#168
s0up4200 merged 1 commit into
developfrom
fix/pushover-typo-161

Conversation

@s0up4200

@s0up4200 s0up4200 commented Jan 24, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Removes the extra colon in the Pushover URL example (pushover://:API_TOKEN@USER_KEY -> pushover://API_TOKEN@USER_KEY)

Fixes #161

Summary by CodeRabbit

  • Documentation
    • Updated the example URL format for Pushover notification service configuration.

✏️ Tip: You can customize this high-level summary in your review settings.

@s0up4200

Copy link
Copy Markdown
Collaborator Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Jan 24, 2026

Copy link
Copy Markdown
✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@coderabbitai

coderabbitai Bot commented Jan 24, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

A typo in the Pushover notification service example URL is corrected by removing an extra colon, changing the format from "pushover://:API_TOKEN@USER_KEY" to "pushover://API_TOKEN@USER_KEY" in the service configuration.

Changes

Cohort / File(s) Summary
Pushover service example URL correction
web/src/api/notifications.ts
Removed extra colon from Pushover example URL format in the SHOUTRRR_SERVICES array

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A colon too many, oh what a sight,
In Pushover's path, it wasn't quite right,
With a whisker and hop, the typo is gone,
The URL now flows, clean and headstrong! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: removing an extra colon from the Pushover notification example URL.
Linked Issues check ✅ Passed The pull request directly addresses issue #161 by removing the extra colon from the Pushover example URL, changing it from 'pushover://:API_TOKEN@USER_KEY' to 'pushover://API_TOKEN@USER_KEY' as requested.
Out of Scope Changes check ✅ Passed All changes are directly related to fixing the reported typo in the Pushover notification example with no extraneous modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Comment @coderabbitai help to get the list of available commands and usage tips.

@s0up4200 s0up4200 merged commit 5fd40ee into develop Jan 24, 2026
10 checks passed
@s0up4200 s0up4200 deleted the fix/pushover-typo-161 branch January 24, 2026 09:22
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.

Typo in notifications.ts

1 participant