acm-uiuc / beats

Beats by ACM
11 stars 19 forks source link

Added Soundcloud Functionality! #62

Closed andrewyang96 closed 9 years ago

andrewyang96 commented 9 years ago

Changes made:

ajmadsen commented 9 years ago

Can you merge the two buttons together into "Play Stream" and handle the back-end based on what the link is? It would reduce UI clutter and standardize the process for adding more streaming services.

andrewyang96 commented 9 years ago

OK, I'll take that suggestion in account.

kevinwang commented 9 years ago

If you go this route you should also revert 6fe031fc9507c29e46014ceb124aacb575013d4d as we will no longer need to update Font Awesome.

andrewyang96 commented 9 years ago

OK, I finished the refactoring. There are merge conflicts, though. I think they're in the config files.