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

task/WG-236: Websocket notifications (#185) #191

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tjgrafft
Copy link
Contributor

@tjgrafft tjgrafft commented Apr 30, 2024

  • task/WG-236: Websocket notifications

  • updating backend for websocket notifications with nginx config changes and socket functions/listeners

  • fixing linting issues

  • linting line spacing

  • whitespace

  • celery app socketio and flask changes for app context

  • Going back to basics and running socketio event directly in the create function

  • fixing linting and emitting default toast message


Overview:

Related Jira tickets:

Summary of Changes:

Testing Steps:

Notes:

* task/WG-236: Websocket notifications

* updating backend for websocket notifications with nginx config changes and socket functions/listeners

* fixing linting issues

* linting line spacing

* whitespace

* celery app socketio and flask changes for app context

* Going back to basics and running socketio event directly in the create function

* fixing linting and emitting default toast message

---------

Co-authored-by: Taylor Grafft <[email protected]>
Co-authored-by: Taylor Grafft <[email protected]>
Co-authored-by: Taylor Grafft <[email protected]>
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.

1 participant