Skip to content

Report all non-fatal import skips via the warnings channel + typed warnings #55

Description

@njakobsen

The feature_update_warnings channel (added in #53) currently only carries the NetworkLink message. The importer silently drops several other non-fatal things:

  • invalid individual placemarks (currently collected into @errors, which raises unless skip_invalid)
  • blank / zero-geometry features (blank_feature?, geom blank)
  • GDAL field-width truncations (e.g. Value … of field Shape_Area … not successfully written)

Proposal: route these non-fatal skips through the same warnings channel so a partial import reports what didn't import and why — not just NetworkLinks. This turns the warning surface into a general "here's what was skipped" report, which is what an uploader actually wants.

Related: consider storing warnings as typed/structured data ({type:, file:, payload:}) instead of pre-formatted strings. That would:

Larger change than #54; prioritize independently.

🤖 Generated with Claude Code

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions