Users should be able to create accounts so they can sign in and claim ownership of permanent rooms.

Checklist:

• User database model
• Password hashing
• Account creation
  • Form to make new User
  • API endpoint to make new User
  • Cryptographically secure random salt generation
  • validate email
  • Client side form validation
    • validate email
    • validate email, username, password length > 0
    • validate retype password
• Rate Limiting
  • Account Creation
  • Account Login
• Show clear errors when log in or registration fails
  • Registration: when email is already used
  • Log in: when email or password is wrong
  • Registration: when username is already used
• User's display names should be separate from what they use to log on
• Make permanent rooms belong to users
• Something to visually indicate whether or not a user is signed in with a permanent account in the user list
• A process for users to claim existing permanent rooms as theirs
• Allow users to change their username (must be unique among registered users)
• Unit tests
  • User log in
  • User registration
  • Permanent rooms should not require the user to be logged in on creation
  • Permanent rooms should set ownerId on creation when the user is logged in

Bugs:

• When a user is in a room, then logs in via logging in to existing account or creating a new account, the name displayed in the user list is not the logged in user's name

Questions that need to be answered:

1. What if users forget their password?
   • For now, tough shit. In the future, password reset email.
2. Should temp users be able to set their name to one taken by a permanent user?
   • Yes, because they will be colored differently

If no response is received from the server, the overlay prevents any interaction with the page and ultimately requires a reload.

Javascript has two enumeration keywords that can be used in for loops: in and of
The former is used to enumerate by index for arrays or keys in objects, and the latter can be used to iterate through an object, such as values of an array.

I propose enforcing the use of for .. of to increase readability/reduce redundancy in code. I am merely using the above as an example; there are many occurrences of the same construct.

for (let r = 0; r < regexs.length; r++) { 
    let matches = res.data.match(regexs[r]); 
    ... 

This can be replaced with:

for (let regex of regexs) {
    let matches = res.data.match(regex);
    ...

The current situation expects the property playbackDuration to be defined in each room. The current workaround is detailed in a comment here:

To fix this, any reference to playbackDuration should be replaced with a reference to currentSource.length

This should be displayed in a bubble next to the "Queue" text

This is caused by the clients not having the video buffered when playback starts.

When a user hits play, the server starts incrementing playbackProgress immediately. When the clients receive the sync message to start playing, they start buffering the video. This usually means at least a second for the client to actually start playing the video. As a result, the user sees the video skip.

There should be some local caching going on so we can minimize requests to the youtube api. It should be handled inside infoextractor.js

In the youtube api, ETags are used to determine if a resource has changed.
https://developers.google.com/youtube/v3/getting-started#etags

I can't think of a reasonable use case where a user would want to add duplicate videos to the queue.

Youtube links in the format https://youtu.be/VIDEO_ID

This doesn't happen in the dev environment, so it creates an inconsistency. No extra slashes should be added to the end

Need logo for branding, to be used in these locations:

• favicon
• navigation bar

Should be in SVG format

Assets needed:

• Logo design
• Logo in minimized SVG
• Placeholder image for videos that don't have thumbnail

There should be a way to change the order of videos in the queue manually.

Referring to the button with the text "Add" that shows under the "Add" tab. This button is no longer needed as a result of the add preview feature.

Only occurs when deployed.

Currently, this endpoint only accepts url. This endpoint should also support adding videos to the queue by specifying service and id, where service refers to what service the video is hosted on, and id is the id of the video on that service.

https://developer.dailymotion.com/tools/
https://developer.dailymotion.com/player/ - Appears to use oEmbed standard, similar to vimeo
https://developer.dailymotion.com/tools/sdks/#sdk-javascript

Player Embed:
https://developer.vimeo.com/player/sdk
API endpoints:
single video: https://developer.vimeo.com/api/reference/videos#get_video
channel: https://developer.vimeo.com/api/reference/channels#get_channel

This probably has something to do with setInterval not being super accurate. Maybe instead of adding 1 second to playback progress every tick, check the actual time delta since the last tick?

Player embed:
https://developers.soundcloud.com/docs/api/html5-widget
API endpoints:
resolve soundcloud url to resource id: https://developers.soundcloud.com/docs/api/reference#resolve
track: https://developers.soundcloud.com/docs/api/reference#tracks
playlist: https://developers.soundcloud.com/docs/api/reference#playlists
user: https://developers.soundcloud.com/docs/api/reference#users

React with 👍 if you are interested in this

Youtube Iframe API: https://developers.google.com/youtube/iframe_api_reference

https://developer.mozilla.org/en-US/docs/Web/API/Fullscreen_API

When given a youtube channel link, the add preview should list out the channel's uploads.

Currently, there is no way to change a room's settings (things like the title, or description).

The current workaround is to have www.opentogethertube.com redirect to opentogethertube.com, but this doesn't work on https because the cert doesn't cover subdomains

Users should have the option to switch the room to "Voting mode." #51 and #52 should be implemented before attempting this feature.

Voting mode should order the videos in the queue according to user's votes.

PR #61

How to reproduce

1. Queue a video
2. Skip the video
3. Queue another video
4. The youtube embed fails to reappear

The thumbnails should show the full thumbnail and no crop out the sides to make it square.

The original togethertube had a chat feature that also served as an event log.

Users could send messages in the chat so other users in the room could see them. This chat box also doubled as an event log, meaning it would show messages like "[USER] joined the room", "[USER] skipped [VIDEO TITLE]", or "[USER] skipped to 3:25".