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.

Sign up for GitHub

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

Jump to bottom

Integrate Notification Micro-Service #238

Open
10 of 13 tasks
leopetter opened this issue Apr 20, 2020 · 2 comments · Fixed by #240
Open
10 of 13 tasks

Integrate Notification Micro-Service #238

leopetter opened this issue Apr 20, 2020 · 2 comments · Fixed by #240
Assignees
@leopetter

Comments

@leopetter
Copy link
Contributor

leopetter commented Apr 20, 2020 •
edited by Quacck
Loading

As a

User

I want

to receive all notifications about my trains at anytime

Because

I am not always logged in and able to receive the notifications over an open websocket

Prerequisites

Tasks

  • Open Pull Request
  • Subscription to correct Kafka queue
  • Connection to UserDB
  • Add redis
  • Save User Notifications in rabbitMQ when logged out
  • Send Notification to the users frontend over websocket when logged in
  • (Possible Addition: contact E-Mail Micro-Service to send notifications to the user)

Acceptance Criteria

  • Documented
  • Merged in dev
  • Linter is okay
  • Tests are written and green
  • Code Review done
  • Reviewed by PO
@leopetter leopetter self-assigned this Apr 20, 2020
@leopetter leopetter mentioned this issue Apr 21, 2020
Merged
6 tasks
@grittaweisheit grittaweisheit linked a pull request May 3, 2020 that will close this issue
Integrate Notification Micro-Service #240
Merged
6 tasks
@leopetter leopetter assigned leopetter and romnn and unassigned leopetter May 6, 2020
@Quacck
Copy link
Contributor

Quacck commented May 11, 2020

Is there a way to see this working?

@romnn romnn closed this as completed May 11, 2020
@romnn romnn reopened this May 11, 2020
@romnn romnn removed their assignment May 11, 2020
@leopetter
Copy link
Contributor Author

RabbitMQ was not suitable for our use case so we are using Redis now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@leopetter leopetter

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Integrate Notification Micro-Service bptlab/cepta
3 participants
@leopetter @Quacck @romnn