Skip to content
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

Issue/ambiguous location 729 #760

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

RizaGhoshMitra
Copy link

Type of change

[728,729,730 ] Bug fix

Changes proposed in this pull request

  • Project name : API Based Weather Report

  • Short description :

No1 - Issue#729 ( Functional)

Resolution
Resolution is now two folded – which created separate user journeys to overcome the bug.
Case1: User can now provide the country name(optional) along with city name , so that application can fetch the weather accurately
image

Case2: Even if user does not provide optional country name , still the outcome now includes the country name.
image

No2 - Issue#730 ( Technical)

image

Checklist

  • [ Y] The code is Python 3 compatible
  • [ Y] I have added relevant comments in my file to make code understandable
  • [ Y] I have added my project in a folder which includes all the necessary files

Copy link
Contributor

github-actions bot commented Jun 6, 2024

Greetings! This is an automated message from GitHub Actions. 🤖
Your pull request has been received and is awaiting for a review by the repository owner or a maintainer. This may take some time, so please be patient.
While you wait, you can continue to work on other issues or pull requests, or explore the project further. Or you can simply relax and enjoy your day.
Thank you for your contribution to this project! You are awesome! ⭐

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.

1 participant