Skip to content

Conversation

@mverteuil
Copy link
Owner

Summary

This PR cleans up deprecated configuration fields and adds an interactive location map component to the settings page.

Changes

Removed Deprecated Fields

  • Flickr integration: Removed flickr_api_key and flickr_filter_email (unused feature)
  • Hardware monitoring: Removed hardware_check_interval, enable_audio_device_check, enable_system_resource_check, and enable_gps_check (redundant monitoring)
  • Privacy threshold: Removed privacy_threshold field (no longer needed)

Renamed for Consistency

  • analysis_overlapaudio_overlap (better naming alignment)

New Feature: Location Map

  • Interactive Leaflet map on settings page
  • Click to set latitude/longitude coordinates
  • Reverse geocoding to show location address
  • Drag marker to adjust position
  • Zoom controls and search functionality

Technical Details

  • Updated config migrations in v2_0_0.py to handle field removals
  • Added location map component using Leaflet.js
  • Enhanced settings route with map integration
  • Added format_coordinates() helper utility

Files Changed

  • 11 files modified
  • 459 insertions, 126 deletions
  • New template: components/location_map.html.j2

Testing

  • Config migrations tested with field removal handling
  • Settings page renders correctly with new map component
  • No breaking changes to existing configurations

Breaking Changes

Config fields removed (were already deprecated/unused):

  • flickr_api_key, flickr_filter_email
  • Hardware monitoring fields
  • privacy_threshold

Migration handles these automatically.

- Remove Flickr integration fields (flickr_api_key, flickr_filter_email)
- Remove hardware monitoring fields (hardware_check_interval, enable_audio_device_check, enable_system_resource_check, enable_gps_check)
- Remove privacy_threshold field
- Add location map component to settings page
- Update config migrations to handle field removals
- Rename analysis_overlap to audio_overlap for consistency
@mverteuil mverteuil force-pushed the feature/config-cleanup branch 3 times, most recently from eb6c4e8 to e132c3c Compare October 21, 2025 03:05
@mverteuil mverteuil force-pushed the feature/config-cleanup branch from e132c3c to c6ad645 Compare October 21, 2025 03:14
@mverteuil mverteuil merged commit 3fc5621 into main Oct 21, 2025
3 checks passed
@mverteuil mverteuil deleted the feature/config-cleanup branch October 21, 2025 03:20
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