Implement Recently Played feature

Satyam1923 / Spring-Music-Player

The Spring Music Player project aims to create a feature-rich music player application leveraging the multiple APIs. This application will provide users with a seamless experience to discover, stream, and organize their favorite music tracks from the extensive library.

https://spring-music-player.vercel.app/

MIT License

66 stars 132 forks source link

Implement Recently Played feature #405

Closed RahulRudra81 closed 2 months ago

RahulRudra81 commented 3 months ago

Describe the feature

Description:

The current music player component does not save the history of played songs. This enhancement involves implementing functionality to store the last 20 songs played in the local storage and displaying them in the recently played section.

Requirements:

Save the details of each song played in local storage.
Maintain a maximum of 20 songs in the history. If the number of songs exceeds 20, remove the oldest entry.
Display the recently played songs in the Recently Played section.
Ensure that song history persists across page refreshes and sessions.

It would look something like this but I will try to make the UI more properly.

Add ScreenShots

Record

[x] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

github-actions[bot] commented 3 months ago

Congratulations, @RahulRudra81! 🎉 Thank you for creating your issue. Your contribution is greatly appreciated and we look forward to working with you to resolve the issue. Keep up the great work! As per GSSoC'24 guidelines the person who raises the issue should be assigned for that issue. But if that person doesn't want to work on that issue the first come fist policy is follwed.

We'd also like to invite you to our Discord server! Join us on discord.

We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

RahulRudra81 commented 3 months ago

@Satyam1923 please assign this issue

Subhajit098 commented 3 months ago

@RahulRudra81 Can you implement this with firebase and not using the local storage ?

RahulRudra81 commented 3 months ago

I am thinking for those who are logged in we can give the feature to save in firebase so for every login they can see their previous history .for users who are not logged in their storage is in local storage. But i will suggest for local storage only because sending data to backend and fetching it will be for no reason. Local storage is a better option as there will be no fetching from backend and it makes the user experience better. Other than that if you want i can save in firebase only but this can only be done for login user because i have to map the history for different users. Because definitely different users will have different history. @Subhajit098

Satyam1923 commented 3 months ago

@RahulRudra81 if we utilize local storage the it will be browser depended not dynamic so I wanted it to be supportive where ever we login our account and we don't need backend to do this i guess only frontend is enough. Local storage for people who are not logged in is good.

Satyam1923 commented 3 months ago

@RahulRudra81 But I guess two people have already raise this issue.

RahulRudra81 commented 3 months ago

@Satyam1923 I have checked nobody recently working on it ... Anyways if I save data in firebase how will i map like which recent belong to which user.

What we can do is who are logged in their recents are stored in firebase making it independent from browser for them. But if you are not logged in just in local storage if their browsers supports they can see recent songs else not. What do you say?

Satyam1923 commented 3 months ago

@RahulRudra81 per user new collection for recent played songs. Ya fine for second point.

RahulRudra81 commented 3 months ago

ok will

github-actions[bot] commented 2 months ago

This issue has been automatically closed because it has been inactive for more than 15 days. If you believe this is still relevant, feel free to reopen it or create a new one. Thank you!