Skip to content
This repository has been archived by the owner on Oct 3, 2023. It is now read-only.

Server and Logging #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Server and Logging #1

wants to merge 2 commits into from

Conversation

rpwnage
Copy link

@rpwnage rpwnage commented Dec 8, 2018

I‘ve added a server to host the files locally, which includes a log of what’s going on and a device specification.

@LinusHenze
Copy link
Owner

Thank you for your PR. However, before merging I have two things to address:

  1. If we're adding a http Server to this Project, it would be great to also integrate a WebSocket Server so the logs can be retrieved. Maybe you could add this?
  2. You did a small change in ready.js, causing a compile error. You probably just did this by accident.

@rpwnage
Copy link
Author

rpwnage commented Dec 9, 2018

Yes, I deleted the ready.js by accident and tried to recreate it. Sorry for that. will be looking into the WebSocket now.

@rpwnage
Copy link
Author

rpwnage commented Dec 15, 2018

Websocket is almost done. Dont close this however, please.

@rpwnage
Copy link
Author

rpwnage commented Dec 15, 2018

what should the websocket return, tho?

@LinusHenze
Copy link
Owner

Nothing. It only needs to listen.

@rpwnage
Copy link
Author

rpwnage commented Dec 18, 2018

fine with that. adding it now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants