Vrp sbox 12 - #35
Merged
Merged
Conversation
* Added loopback * Added MongoDB test * Added MongoDB config for CircleCI * Deleted watcher for watch and dev scripts * Added exit functionality to test script
* Adds boot script to add Admin on server start * Disables server authentication for loopback and completes boot script
* Adds initial route configuration and react router * Updates file structure, fixes pathing errors * Adds react router, configures routes, adds a preliminary dashboard for admin
* fixed user relation model so there is one queue for every user * created remote method for getplaylist that returns a list of song tracks currently on the playlist * created updatePlaylist remote method that works to get current playlist > remove current playing song > add new song > update spotify playlsit to reflect those changes. Next, I need to get it to add default songs when the playlist only has one song left, and I need to track last played songs, and I need to make this remote method a post request to update the queue field in the queue model * simplifed updatePlaylist by using getPlaylist within it * fixed authorization issues so others can authorize by going to localhost/spotify
* Comments out socket.io requests, fixes AdminContainer, adds HashRouter * Fixes router to render Admin Dashboard
* adds styling using grid and username and password state working * adds login test and styling * updates gitignore to exclude dist folder * Removes fonts from src folders * Delete bundle.js
* adds back fonts * adds font folder
* Adds initial route configuration and react router * Adds initial route configuration and react router * Updates file structure, fixes pathing errors * Fixes file structure, again * Fixes some things * Rebased * Fleshes out CSS for Dashboard * Adds initial styling for Admin dashboard * Adds proper text styling for default queue * Extends design improvements from default queue to current queue * Adds initial style for play button; adds action and reducer * Incorporates static play button into currentSong component * Removes now-defunct play button
fixes admin dashboard errors
* Added a remote method that checks if a song exists in database. If not, it makes Api call to spotify and adds the song to database, otherwise it returns the song from database * Changed test name to getSong functionality
* Minor routing issues * Fixes routing
* created updatePlaylist remote method that works to get current playlist > remove current playing song > add new song > update spotify playlsit to reflect those changes. Next, I need to get it to add default songs when the playlist only has one song left, and I need to track last played songs, and I need to make this remote method a post request to update the queue field in the queue model * simplifed updatePlaylist by using getPlaylist within it * fixed authorization issues so others can authorize by going to localhost/spotify * updatePlaylist remote method is now a functioning put request that updates the songId array of that queue * added default songs to the queue. made it so queue populates with song id. added last play variable and conditions back. * fixed last played to update in queue * returned error if duplicates are added. fixed it so last played updates the queue and actually works. * fixed it so all the edge cases work * added playlist tests
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
VRP-SBOX-12 was initially meant to represent the search for songs, but now has change to represent SBOX-4, which is to add socket.io for Loopback and ensure each client can broadcast to each other within each room.