[LANDGRIF-1625] fixes calling /h3data endpoint wrongly when layer is raster

[agnlez]

This pull request includes several changes to improve the handling of contextual layers and their data fetching in the analysis visualization components. The most important changes include the addition of a new dependency, updates to the contextual layer components, and the refactoring of hooks for better performance and maintainability.

Dependency Addition:

• Added @lukemorales/query-key-factory to client/package.json to manage query keys more efficiently.

Contextual Layer Components:

• Enhanced the Loading component in category-layer/component.tsx to include specific class names for better styling.
• Updated preview-map/component.tsx to use new hooks and handle contextual layers more effectively, including adding the MapboxRasterLayer component and improving the status management of layers. [1] [2] [3]
• Removed unused imports and hooks from contextual-legend-item/component.tsx to clean up the code. [1] [2] [3]

Hook Refactoring:

• Refactored contextual.ts to remove the useAllContextualLayersData hook and update the useH3ContextualData hook to use the new query key store. [1] [2] [3] [4]
• Updated index.ts to align with the new query key store and improve data fetching logic. [1] [2]
• Removed the useContextualLayer hook from contextual.ts as it is no longer needed.

Query Key Store:

• Introduced a new query key store in querykey-store.ts to manage query keys centrally and improve the consistency of data fetching across the application.

Application Initialization:

• Updated _app.tsx to use a centralized query client configuration by importing getQueryClient from the new query key store. [1] [2]

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
landgriffon-client	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Feb 25, 2025 1:23pm

2 Skipped Deployments

Name	Status	Preview	Comments	Updated (UTC)
landgriffon-cookie-traceability	⬜️ Ignored (Inspect)	Visit Preview	💬 Add feedback	Feb 25, 2025 1:23pm
landgriffon-marketing	⬜️ Ignored (Inspect)	Visit Preview		Feb 25, 2025 1:23pm

[alexeh]

U the man @agnlez thanks.

It works well, I haven't found my way to any weird behaviors, except this:

But this seems to be some internal react/next/whatever magic related to query key/caching or something that I don't understand. It does not seem to hit the backend so I leave up to your expertise what to do regarding this.

However, please, for the love of that's holy, add some tests before merging this. We can pair up to think about this. Meanwhile I will block it

@monro93 what are your thoughts? Are you up add some tests to this?

[agnlez]

@alexeh those requests are done by the map (specifically the maplibre library) when fetches tiles. Sometimes, if the user moves quickly in the map, some requests are cancelled because they are not within the viewport anymore. This is an expected behavior, nothing harmful. (There might be other behaviors that might trigger this)