Skip to content

Conversation

@rituraj12797
Copy link
Contributor

Developer Checklist

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

Summary

Added the weather app project

Screenshot

image

Live Project Link

https://rituraj12797.github.io/weatherApp/

@rituraj12797
Copy link
Contributor Author

@thinkswell i have added the pr for the issue #609

@rituraj12797
Copy link
Contributor Author

kindly review the pr and merge if you liked it 👍

@PBJI
Copy link
Collaborator

PBJI commented Oct 8, 2023

@rituraj12797
You should considering adding your project in the pre-existing WeatherApp folder and not weather app folder.

And consider adding README.md with

  1. Summary about your project
  2. How to setup and use
  3. Add the screenshots in it.

Lastly all your project files, including the README.md should be inside another folder called rituraj12797

javascript-min-projects
|           
-------- WeatherApp
           |
            ----- rituraj12797
                   |
                    -------- Your project files and folders
                   |
                    --------- README.md

Refer to How to raise the PR for more information

Thank you.

@rituraj12797
Copy link
Contributor Author

working on it

@rituraj12797 rituraj12797 closed this by deleting the head repository Oct 8, 2023
@rituraj12797
Copy link
Contributor Author

@PBJI done added another pr please review

@rituraj12797
Copy link
Contributor Author

pr number #690

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants