Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: implemented pusher service for real time notifications. #1302

Open
wants to merge 6 commits into
base: dev
Choose a base branch
from

Conversation

edwinedjokpa
Copy link

@edwinedjokpa edwinedjokpa commented Feb 28, 2025

Pull Request

Description

This PR adds a Pusher service to handle real-time event triggering and its corresponding unit tests.

Related Issue

#1289

Fixes #(issue)

New Feature

  • feat: New feature
  • fix: Bug fix
  • docs: Documentation updates
  • style: Code style/formatting changes
  • refactor: Code refactoring
  • perf: Performance improvements
  • test: Test additions/updates
  • chore: Build process or tooling changes
  • ci: CI configuration changes
  • other:

How Has This Been Tested?

  • Unit tests
  • Integration tests
  • Manual tests

Test Evidence

pusher_test

Screenshots (if applicable)

Documentation Screenshots (if applicable)

Checklist

  • My code follows the project's coding style
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published
  • I have included a screenshot showing all tests passing
  • I have included documentation screenshots (if applicable)

Additional Notes

This implementation ensures proper handling of Pusher events, including logging success messages and throwing errors when necessary. The unit tests validate successful event triggering and error handling scenarios.

AdeGneus
AdeGneus previously approved these changes Feb 28, 2025
@incredible-phoenix246
Copy link
Contributor

Reason for this PR?

@edwinedjokpa edwinedjokpa changed the title Feat/pusher service feat: implemented pusher service for real time notifications. Feb 28, 2025
@edwinedjokpa
Copy link
Author

Reason for this PR?

New feature for real time notifications with pusher as my issue.

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