{"push_id":2744323981,"size":1,"distinct_size":1,"ref":"refs/heads/master","head":"20cc4fe1e60731f2b56fb7d3c9c8fd19530cb24c","before":"11de473a7032b78e34e0c84f419941973397001b","commits":[{"sha":"20cc4fe1e60731f2b56fb7d3c9c8fd19530cb24c","author":{"name":"Alex Dinh","email":"26d5032d75f5771a8d0b7628f54f95e610b805aa@gmail.com"},"message":"Playlist switching now works\n\nissue resolved: react-router-dom scope\r\n\r\nROUTER INSIDE ANOTHER ROUTER DOESN'T WORK, ROUTES OUT OF SCOPE\r\n- MemoryRouter inside UserPlaylists.js was making it out of scope of the MemoryRouter in App.js\r\n\r\nNeed to update component when prop changes:\r\n- componentWillReceiveProps is now legacy, considered buggy and unsafe coding practice\r\n- React dev team encourages use of memoization: https://reactjs.org/blog/2018/06/07/you-probably-dont-need-derived-state.html#what-about-memoization\r\n- easy solution: componentDidUpdate() (CURRENTLY IN USE)\r\n\r\nPlaylist switching now works!! :)\r\n\r\nStyling is annoying","distinct":true,"url":"https://api.github.com/repos/alex-dinh/medlify/commits/20cc4fe1e60731f2b56fb7d3c9c8fd19530cb24c"}]}
{"actor":{"display_login":"alex-dinh"}}