Skip to content

Add Django Channels #49

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

Open
wants to merge 17 commits into
base: main
Choose a base branch
from
Open

Add Django Channels #49

wants to merge 17 commits into from

Conversation

tjquillan
Copy link
Contributor

This PR adds live updating functionality to the coach and student pages

@tjquillan tjquillan requested a review from tjzetty November 15, 2022 19:16
@tjzetty
Copy link
Contributor

tjzetty commented Nov 15, 2022

Don't know what's going on here but the websocket keeps disconnecting on the coach and student pages. Getting a connection refused error after HANDSHAKING:
ConnectionRefusedError: [Errno 61] Connect call failed ('127.0.0.1', 6379)

Do either of you guys know what could be going on?

@tjquillan
Copy link
Contributor Author

You need to run docker compose up -d like with MySQL

@tjzetty
Copy link
Contributor

tjzetty commented Nov 15, 2022

Alright cool, didn't know we had docker on this version but that makes sense. Is the student page broken cause we don't have real users set up yet?

@tjquillan
Copy link
Contributor Author

Yeah, if you want to test it for now you can change

self.user = self.scope["user"]
to a hardcoded user from your database.

@tjzetty
Copy link
Contributor

tjzetty commented Nov 16, 2022

Okay last question, how are you passing the table through right now, I noticed that the grabQuestions() method in views.py is broken in this build and I imagine it is related. I see the websocket JS code but where in the backend is it grabbed. Everything else is looking really good though, good job guys.

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