Fix for multiple Lo-fi and meditate songs playing at once

rojin254 commented 9 months ago

Related Issue

Closes: #122 Makes only one song play at a time under Lo-Fi and Meditate tabs

Describe the changes you've made

I added some props to the card component which keeps track of the current song playing and also checks if multiple songs needs to be played. I added a useEffect hook to toggle playing state of previously playing song cards depending on the current Song tracker state ( Provided the multi prop is set to false)

Type of change

What sort of change have you made:

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Ran npm server and tested it on chrome and edge browsers

Checklist:

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

Screenshots

vercel[bot] commented 9 months ago

Someone is attempting to deploy a commit to a Personal Account owned by @Utkarshn10 on Vercel.

@Utkarshn10 first needs to authorize it.

rojin254 commented 9 months ago

done @Utkarshn10

Utkarshn10 commented 9 months ago

Thanks @rojin254 for contributing to Focusly :).

Utkarshn10 commented 9 months ago

Hey @rojin254 this PR isn't merged. Could you please raise another PR.

rojin254 commented 9 months ago

oops my bad I thought the PR was merged and closed. I have raised a new PR #126

Utkarshn10 / Focusly