Skip to content
This repository has been archived by the owner on Oct 15, 2023. It is now read-only.

Created weather monitoring app using flutter #137

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

2021eo3ar
Copy link

Description

I have made a weather monitoring app using flutter and used Open weather map API to fetch the weather report of a location.

How Has This Been Tested?

I have made a flutter app of weather monitoring and tested it in FlutLab.io and android studio.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@gitguardian
Copy link

gitguardian bot commented Oct 13, 2023

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secret in your pull request
GitGuardian id Secret Commit Filename
- OpenWeatherMap Token fefec5f Weather monitoring app using flutter/lib/services/weather.dart View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

Our GitHub checks need improvements? Share your feedbacks!

@2021eo3ar
Copy link
Author

Hey @tanmoy27112000, I have submitted the PR and put a lot of effort in making the app, Can you please review it as soon as possible.

@tanmoy27112000
Copy link
Owner

attach screenshots

@2021eo3ar
Copy link
Author

Weather screenshots

Weather Screenshot 2

Working of app

  1. first it needs location permissions.
  2. After all permissions it automatically detects the Weather of current location.
  3. by clicking on city icon we can search for the weather condition of a specific city.
  4. and by pressing on another kite icon it re-renders the weather conditions of current location.

@2021eo3ar
Copy link
Author

Hey @tanmoy27112000 , Please merge my PR.

@tanmoy27112000
Copy link
Owner

Please add 3-4 ui screen to be considered as Valid PR

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants