statkus / ambi Goto Github PK

The button should indicates if a new room will be created or if the user will join an existing one, depending on the search input.

The server could retrieves this information when registering a new client to a room to not reset the checkboxes values to the default value (player check, sync uncheck).

It would bé useful (girlwise) to sée in the vidéo list which vidéo display thé lyrics if thé song.

Searching a song using the character "&" results in ignoring the following words because what is after the "&" is considered as different parameter in the request.

Song liked in one room cannot be liked in another one. This is because the song id is the key of the likes table, the database should be modified to handle that.

Add the possibility to add all videos from a Youtube playlist.

Some Room members are already protected by other mutex (playlist, current video) is this sufficient ? This global mutex can block all the server if a client request raises a not-handled exception.

When a Youtube video link with an incorrect video ID or a video not embeddable is requested to the search bar, the server will raise an exception because the vector containing the video corresponding to the request will be empty. This exception will prevent the room mutex to be released and will block all other room requests. The exception must be handled to prevent that.

Add a button to add a random song from likes list to the current playlist.

• First step: only support direct soundcloud link in search bar
• Second step: add support for search terms

This is related to the project https://github.com/Statkus/json-ada.

Pass Youtube API key and server address and port as arguments to the executable instead of using files.

The next video button allow to change the current video of all sync clients, it could be better to add a poll system so that the current video changes only if a given percentage of the sync clients want to. The current number of votes must be displayed.

It might makes the code more readable and easier to maintain.

If the title is too long the end is cut. This is especially bad when using the app on a small screen.

See AUnit.

When loading the Ambi, the focus should be given to the search bar so we can directly type the name of the room we are looking for without having to click on it first.

In order to avoid adding stupid Youtube videos that last forever and more, the search result list should display the duration of every entry in addition to the cover picture and the title.

Nan mais merde, c'est pas possible qu'en 2021 on n'ait aucun moyen de voir si on s'apprête à ajouter à la liste de lecture un roman fleuve d'une heure trente. Ça ne peut plus continuer comme ça.

For example the Youtube video LQZLPV6xcHI has a thumbnail bigger than usual (https://i.ytimg.com/vi/LQZLPV6xcHI/default.jpg). The thumbnails size should be forced.

The JS part of room.thtml should be in a separate file for readability.

Maybe a logging package should be created to manage that instead of directly using Put_Line everywhere.

The vertical alignment must support multi lines title.