Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Final sprint close for boba & vata #39

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

Final sprint close for boba & vata #39

wants to merge 28 commits into from

Conversation

anthonyvalera
Copy link
Contributor

No description provided.

vancepope and others added 27 commits October 25, 2018 16:28
* 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
* 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
* adds font folder

* fixes vw of login button

* finishes styling so that it looks the same on all monitor sizes

* updates bundle

* updates bundle
* Added update type and search input functionality to redux

* Add handleSearch method and deleted uri search capability

* added getSearch action

* Added socket.io

* Add handleSearch method and deleted uri search capability

* added getSearch action

* Added socket.io

* Updated socket.io

* Add handleSearch method and deleted uri search capability

* added getSearch action

* Added socket.io

* Updated socket.io

* Merged conflicts

* Reviewed changes

* Changed app.jsx

* Committing before rebase

* Added socket.io custom room capability
* Adds key for song queue map function on admin dashboard

* Admin can now add song from admin dashboard using URI

* Adds class names to buttons for testing purposes
* Adds initial play button, centered correctly

* Removes bundles

* Adds Actions and Reducer to CurrentSong

* Adds depth to button, fixes triangle

* Play/Pause Switch fixed

* Partially adds loopback implementation for SongActions

* Remote method functional; requires connection to front-end

* Adds play and pause functionality

* Fixes error in Song Reducer

* Finalizes playback button with incomplete testing foundation

* Adds completed testing. Ticket completed

* Removes commented out code

* Fixes accidental removal

* Fixes testing environment, removes console log; ready to push to Dev

* Adds small comment for future development
* updated front end search bar. started work on axios calls for the search by artist

* finished the db query and mapping out the results. currently working the functionality of two buttons to add the uri to the queue

* finished the db query and mapping out the results. currently working the functionality of two buttons to add the uri to the queue

* changed the function to add song to queue

* updated the artist search results button and the song search button.

* finished SBOX-14. fixed search and then add songs to queue.
client-old/package.json Outdated Show resolved Hide resolved
common/models/queue.js Outdated Show resolved Hide resolved
common/models/queue.js Outdated Show resolved Hide resolved
common/models/queue.js Outdated Show resolved Hide resolved
common/models/queue.js Outdated Show resolved Hide resolved
tests/playlist.spec.js Outdated Show resolved Hide resolved
tests/playlist.spec.js Outdated Show resolved Hide resolved
tests/playlist.spec.js Outdated Show resolved Hide resolved
tests/server.spec.js Outdated Show resolved Hide resolved
tests/song.spec.js Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants