Skip to content

Conversation

@gurjeetsinghvirdee
Copy link
Contributor

@gurjeetsinghvirdee gurjeetsinghvirdee commented Oct 9, 2025

📖 Overview

This PR implements a lightweight playlist recommendation feature that suggests curated Spotify playlists based on current weather conditions and optional user mood selection.

✨ Features Added

🌦️ Weather-to-Music Mapping

  • Rain 🌧️ → Cozy/Jazz playlists (Acoustic, Lofi, Jazz Vibes)
  • Clear ☀️ → Energetic/Pop playlists (Happy Hits, Mood Booster)
  • Snow ❄️ → Calm/Classical playlists (Peaceful Piano, Winter Acoustic)
  • Thunderstorm ⛈️ → Intense/Electronic playlists (EDM, Beast Mode)
  • Clouds ☁️ → Mellow/Contemplative playlists (Evening Acoustic)
  • Drizzle 🌦️ → Soft/Soothing playlists

😊 Optional Mood Selector

Users can optionally refine playlist recommendations by selecting their current mood:

  • 😊 Happy
  • 😌 Relaxed
  • 🧠 Focused
  • ⚡ Energetic
  • ✨ Calm

This approach respects that SkyBuddy is currently a website where users visit briefly to check the weather, not to stay and listen to music in the browser.

🖼️ Screenshots

image

📹 Other Attachments

Recording.2025-10-10.032616.1.mp4

- Implement weather-to-music mapping with curated Spotify playlists
- Add playlist card component with external Spotify links
- Create optional mood selector for personalized recommendations
- Integrate playlists into city and dashboard pages
@gurjeetsinghvirdee
Copy link
Contributor Author

Hey @SrinjoyeeDey 😅, just created a PR. Review whenever you get a chance!

@SrinjoyeeDey
Copy link
Owner

SrinjoyeeDey commented Oct 10, 2025

Heyy @gurjeetsinghvirdee , I just found this from another website and absolutely looks cool , well I would like to have a dedicated page for it , Can you do that . Add the features and the one you already did , and make sure users can adjust the sound as well , This will make it more interesting . When someone will click the music the interface will look the like this and they will have a choice to add more musics .
image

and make sure to keep this part
image

Please confirm whether you can proceed with it ?

@SrinjoyeeDey
Copy link
Owner

image can you please resolve this conflict @gurjeetsinghvirdee

@gurjeetsinghvirdee
Copy link
Contributor Author

Hey @SrinjoyeeDey 👋
Rn when a user clicks on a playlist, it opens directly on Spotify. But since you want a separate page for it do you mean the music should play directly on our website instead of redirecting to Spotify? Because then all those controls (like play/pause, volume, etc.) would make sense.

Also, for the add more music part could you please clarify whether you want users to add music links (like Spotify/YouTube URLs) or upload audio files (drag & drop)?

@gurjeetsinghvirdee
Copy link
Contributor Author

image can you please resolve this conflict @gurjeetsinghvirdee

This is not a conflict, thats a IDE intellisense issue. I also tried to fix that but nothing works but that not causing any error on the website

@SrinjoyeeDey
Copy link
Owner

image image But its causing my website to crash

@SrinjoyeeDey
Copy link
Owner

Actually @gurjeetsinghvirdee , there is some problem regarding HealthRecommendations , the PR you raised previously . Can you update your local repo again and the check the issue , since its related to your previous PR I think you can resolve it seamlessly

@gurjeetsinghvirdee
Copy link
Contributor Author

image image But its causing my website to crash

I'll try but that thing doesn't show up when I test the code

@gurjeetsinghvirdee
Copy link
Contributor Author

Okay I will go through again will check what causing the error.

@SrinjoyeeDey
Copy link
Owner

Just resolve the errors , if it works fine then raise the PR , I will merge it . And about a dedicated page for the playlist , I will raise a separate issue for that and you can work there

@gurjeetsinghvirdee
Copy link
Contributor Author

gurjeetsinghvirdee commented Oct 10, 2025

So what about this PR? Rn what other changes needed here.
And for that bug it will take time coz powercut in my area from 10 to 3

@SrinjoyeeDey
Copy link
Owner

@gurjeetsinghvirdee after you will resolve the error , you will certainly make some changes in files and when I will merge this PR , eventually the problems on those files will get overwritten by your changes and It will solve the problem

@gurjeetsinghvirdee
Copy link
Contributor Author

Then that issue you created for the solving the bug doesn't make sense as 2 of issues solving on 1 PR.
Not a issue on my side will fix it

@SrinjoyeeDey
Copy link
Owner

  1. Pull the latest changes from the main branch into his PR branch:

git checkout his-branch
git pull origin main

or if using rebase:

git fetch origin
git rebase origin/main

  1. Resolve any conflicts that come up.

  2. Push the updated branch to the PR @gurjeetsinghvirdee


@SrinjoyeeDey
Copy link
Owner

Yess @gurjeetsinghvirdee , after your PR works well , I will close that issue I raised, don't worry

@gurjeetsinghvirdee
Copy link
Contributor Author

gurjeetsinghvirdee commented Oct 10, 2025

Hey @SrinjoyeeDey
Can you review again? I fixed that list error

Recording.2025-10-10.161250.1.mp4

@SrinjoyeeDey
Copy link
Owner

Working absolutely fine @gurjeetsinghvirdee , Thanks for so much help , Merged

@SrinjoyeeDey SrinjoyeeDey merged commit bd53071 into SrinjoyeeDey:main Oct 10, 2025
@SrinjoyeeDey
Copy link
Owner

@gurjeetsinghvirdee , I would like to open another issue regarding enhancement of playlist and to add additional features , you may or may not work on it But I am truely grateful for your amazing patience and contribution

@gurjeetsinghvirdee
Copy link
Contributor Author

@gurjeetsinghvirdee , I would like to open another issue regarding enhancement of playlist and to add additional features , you may or may not work on it But I am truely grateful for your amazing patience and contribution

Well I want to work! if you assign me it will be grateful already loving your project

@gurjeetsinghvirdee gurjeetsinghvirdee deleted the mood-based-playlist branch October 10, 2025 14:11
@gurjeetsinghvirdee
Copy link
Contributor Author

@SrinjoyeeDey
I think you forgot to attach a label here!
And if this project is not just for hactoberfest and will be active in the future, I will be contributing to this. I mean, I like this project, and I find that most of the project goes dead after the program is over.

@SrinjoyeeDey
Copy link
Owner

Nope @gurjeetsinghvirdee , I will make it open for open-source until it becomes a full-fledged application and will give credits to all the contributors for its success

@gurjeetsinghvirdee
Copy link
Contributor Author

@SrinjoyeeDey
That's nice, will be happy to contribute to this repo.
And whenever you make that new issue, assign me. I will ask to be assigned if you forgot.

@SrinjoyeeDey
Copy link
Owner

#20 @gurjeetsinghvirdee , you can contribute there

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TypeError / Property 'list' does not exist on data returned by weather hooks Issue: Mood-Based Weather Playlists 🎵🌦️

2 participants