Skip to content
New issue

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

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

Already on GitHub? # to your account

socket.io integration to lectures #236

Closed
wants to merge 1 commit into from
Closed

socket.io integration to lectures #236

wants to merge 1 commit into from

Conversation

karinocheretny
Copy link
Contributor

Socket.io integration to lectures:

To see changes in the server as a student:
log in as a student, and connect to any lecture by clicking "view lecture". The server will log out that a user has connected to the lecture.

To see changes as a teacher:
log in as a teacher, click "view sessions", click "lectures" in the menu, then click "edit lecture". The server will log out that a teacher had connected. (For future teams might be helpful to change this button name to connect to lecture, etc.)

@karinocheretny karinocheretny requested review from a team May 27, 2024 00:49
Copy link
Member

@nilsstreedain nilsstreedain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After installing socket.io-client in /client/package.json, I'm still having issues running the site locally. It occasionally appears to load infinitely, however I've gotten the following error to display a few times:

Uncaught runtime errors:
×
ERROR
undefined is not an object (evaluating 'e.response.status')
@http://localhost:3000/static/js/bundle.js:10493:19

I've tried fully wiping the repo and any docker volumes from my machine but the problem persisted.

import Notice from '../components/Notice'
import { Button, Card } from "react-bootstrap"
import useLectureQuestions from "../hooks/useLectureQuestions";
import useCourse from "../hooks/useCourse";
import useLectures from '../hooks/useLectures';
import QuestionCard from '../components/QuestionCard';

import io from 'socket.io-client';
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

socket.io-client appears to be imported, but is not added to the client package.json causing compilation to fail.

@n8srumsey
Copy link
Contributor

Architecture design has changed, and this implementation of socket.io for features will no longer be valid. Closing PR.

@n8srumsey n8srumsey closed this Nov 17, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants