Skip to content

Fix/dynamic user location#12

Open
Apprentice2907 wants to merge 3 commits into
CCExtractor:mainfrom
Apprentice2907:fix/dynamic-user-location
Open

Fix/dynamic user location#12
Apprentice2907 wants to merge 3 commits into
CCExtractor:mainfrom
Apprentice2907:fix/dynamic-user-location

Conversation

@Apprentice2907
Copy link
Copy Markdown

Description

This pull request replaces the previously hardcoded New Delhi coordinates with the user’s real-time current location for location-based alarm conditions. The implementation uses the existing fl_location package and handles location permissions gracefully, with a fallback location when access is unavailable.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactor (code improvement/cleanup without logical changes)
  • UI/Design change

Proposed Changes

  • Removed hardcoded latitude and longitude values used for location-based conditions
  • Added dynamic fetching of the user’s current location using fl_location
  • Implemented proper permission and service availability checks
  • Added a fallback location when location access is denied or unavailable
  • Ensured the map and location picker initialize with the user’s current location

Fixes #6

Screenshots

Not applicable (logic change only).

Checklist

  • Tests have been added or updated to cover the changes
  • Documentation has been updated to reflect the changes
  • Code follows the established coding style guidelines
  • All tests are passing

@Apprentice2907
Copy link
Copy Markdown
Author

Hi 👋
I’ve opened a PR that replaces the hardcoded New Delhi coordinates with the user’s real-time current location using the existing fl_location package.

The change includes proper permission handling and a safe fallback when location access is unavailable, without modifying any UI or dependencies.

Please let me know if any adjustments are needed. Thanks!

@Abhishek14104
Copy link
Copy Markdown
Collaborator

Thanks for the PR! Please add a demonstration video of the working app with the log outputs with your PRs

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.

Enhancement: Replace hardcoded location with user’s current location

2 participants