Skip to content

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

@njakobsen

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
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions