Skip to content

Conversation

@kunal-511
Copy link
Contributor

@kunal-511 kunal-511 commented Oct 10, 2023

Developer Checklist

  • Followed guidelines mentioned in the readme file.
  • Followed directory structure. (e.g. ProjectName/{USERNAME}/...yourfiles)
  • Starred ⭐ the Repo (Optional)

Summary

I have created a weather app using Rapid Api. Made this project using VanillaJs.

Screenshot

image

Live Project Link

https://kunal-511.github.io/weather_app/

@kunal-511
Copy link
Contributor Author

Please merge this to main branch

Copy link
Collaborator

@PBJI PBJI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hi @kunal-511 , change your project folder name to WeatherApp/kunal-511

@PBJI PBJI added the changeRequested The requested changes should be adhered to before further review or approval. label Oct 11, 2023
@kunal-511
Copy link
Contributor Author

I have changed Please merge under HacktoberFest Label

@PBJI
Copy link
Collaborator

PBJI commented Oct 12, 2023

@kunal-511 This repository is excluded from hacktoberfest because it violates their rules and values. However your contribution is still valued and comes under open-source.

Thank you so much,
Purshotam Bohra

@PBJI PBJI merged commit 9c7bf82 into thinkswell:master Oct 12, 2023
@kunal-511
Copy link
Contributor Author

No Problem If it is not under Hacktober Fest

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

Labels

changeRequested The requested changes should be adhered to before further review or approval.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants