-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Integrate OpenWeatherMap API #12
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left a comment on the GeoMap component.
I think you were not able to commit all your changes from your local end (?) since the implementation for the API requests are missing.
Also, you can format your changes with mvn formatter:format
to satisfy the PR check
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some comments around on the edited files. Aside from that:
- For the formatter check, you can fix them with
mvn formatter:format
. - For the
yourpackage
folder, I think we can rename it with a more fitting name such asapi
(?) folder instead. - The previous comments & java docs on some parts were removed and are still left unchanged, I do not know if this is intentional.
P.S. my DMs are open anytime, ask for any Qs or help lang if needed 🫡
all tasks given... i might have missed to pr some other changes cuzz im having trouble making PR just DM if found some error or mistake, our net is now good so i can fix anything ASAP if needed