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

[INV-3779] Use the LocalDB instances for Recordset CacheMetadata instead of Redux #3805

Merged
merged 29 commits into from
Jan 20, 2025

Conversation

LocalNewsTV
Copy link
Collaborator

@LocalNewsTV LocalNewsTV commented Jan 16, 2025

Overview

Important

Please do extra testing on IAPP Whats Here functionality when testing this due to the new Endpoint playing a role in it :)

Main highlights

Misc

  • update Error messages in Caching to show names of caches failing
  • [SQL] Adjust caching of Activities to more reflect IAPP (set GeoJSON shapes to include the label Data). Previously grabbed POST-cache
  • increase cases where we call handle_MAP_INIT_STATE
    • Fixes issues where newly cached Layers aren't available offline (same session)
    • Fixes issues where opening the App offline, then going online makes IDLists not available to uncached records
  • Initialize RecordCache Singleton at app load
    • Fixes issue where calling SQL caches to gather IDLists for Records causes a soft error of "cacheDB not found" due to n init calls

Tip

Primary Code duplication stems from creating a BBox endpoint for the IAPP Records.

@LocalNewsTV LocalNewsTV changed the title 3779 use local database over redux [INV-3779] Use the LocalDB instances for Recordset CacheMetadata instead of Redux Jan 16, 2025
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)
19.5% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@LocalNewsTV LocalNewsTV linked an issue Jan 20, 2025 that may be closed by this pull request
2 tasks
@LocalNewsTV LocalNewsTV merged commit fba8798 into dev Jan 20, 2025
6 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant