Skip to content

Cleanup: Remove Leaflet and replace it's types #35

Closed

Description

@Alessandro100

With the migration to Nextjs, leaflet has been replaced in favor of maplibre

the only remains of it in our code base is used as types

import { type LatLngExpression } from 'leaflet';

remove this and find an alternative

bonus: reorganize the datastructure of how we store our polygons for less transformations

Activity

  1. ianktc commented on Jan 29, 2026

    @ianktc
    Contributor
  2. Alessandro100 commented on Jan 29, 2026

    @Alessandro100
    ContributorAuthor

    It's the same concept, but different data structure. See image

    Image

    The real data structure resembles leafletLngLat, there is a possibility that we could reorganize it to fit the new maplibre one, but ideally not (if existing functionality works, we shouldn't touch it). Requires some research

  3. added
    tech-debtShortcuts, workarounds, or code that needs refactoring or cleanup
    on Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

tech-debtShortcuts, workarounds, or code that needs refactoring or cleanup

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions