2nfm
2nfm copied to clipboard
codysherman
Enable text chat
@andmarek has completed the code that talks to the backend, so this is just waiting on updated UI from @bfrisch1 @AdarBS
@samtheprogram says he's received the frontend UI from @AdarBS. @JohnmarkBeaty expressed interest in building this once that has happened.
Post-hackathon-mayhem, I am not sure where @andmarek's pushed code that talks to the backend is -- if we don't hear back from him, we can re-implement as RTCMultiConnection provided most of the heavy lifting here anyway.
In the event we need to re-implement some of this logic, the original implementation from the hackathon utilized RTCMultiConnection's send method.
As far as UI goes, see the branch chat-styles or chat-room-js for what was done by @AdarBS and @bfrisch1.
The two branches diverged, so there are some missing commits from @bfrisch1 not present in chat-room-js, but they may not be necessary as @AdarBS's latest changes in chat-room-js, which came later, was explicitly for polish.
Adar's work goes far beyond mine, I can check and try and push anything that stands out (sorry about that) but I doubt it's important. He took my branch and ran with it.
On Tue, Jun 16, 2020 at 1:48 PM Sam Moore [email protected] wrote:
As far as UI goes, see the branch chat-styles or chat-room-js for what was done by @AdarBS https://github.com/AdarBS and @bfrisch1 https://github.com/bfrisch1.
The two branches diverged, so there are some missing commits from @bfrisch1 https://github.com/bfrisch1 not present in chat-room-js, but they may not be necessary as @AdarBS https://github.com/AdarBS's latest changes in chat-room-js, which came later, was explicitly for polish.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/codysherman/2nfm/issues/56#issuecomment-644976604, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADCHIMNECVQZNA2VVF5GU2DRW7EARANCNFSM4IZEFCIA .
