Skip to content

added authoritative store in params, entity-org creation, and plugin …#95

Merged
pooleycodes merged 5 commits intomainfrom
entity-org-and-plugin-errors
Feb 17, 2026
Merged

added authoritative store in params, entity-org creation, and plugin …#95
pooleycodes merged 5 commits intomainfrom
entity-org-and-plugin-errors

Conversation

@pooleycodes
Copy link
Copy Markdown
Contributor

@pooleycodes pooleycodes commented Feb 6, 2026

The params for Add Data task now include a 'authoritative' param, this is mostly used as a storage param currently. Every time add data is run, entity-organisation csv will be created and it is up to the front end to use this data or not.

This PR also gives more detail on if a plugin has failed (not guaranteed to catch)

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Related Tickets & Documents

QA Instructions, Screenshots, Recordings

  • Check entity-org csv is created in add data task.
  • Give a faulty arcgis url to see error detail

Added/updated tests?

We encourage you to keep the code coverage percentage at 80% and above. Please refer to the Digital Land Testing Guidance for more information.

  • Yes
  • No, and this is why: Minor changes
  • I need help with writing tests

@pooleycodes
Copy link
Copy Markdown
Contributor Author

also added to this a bug detected where the url not the url hash was being passed to the pipeline, causing column mappings to fail

Copy link
Copy Markdown
Contributor

@eveleighoj eveleighoj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think just leave the lint target as is, it should be used in the CIto check black has been run so changing it to format the code is dangerous

@pooleycodes pooleycodes merged commit 687c8e9 into main Feb 17, 2026
2 checks passed
@Ben-Hodgkiss Ben-Hodgkiss deleted the entity-org-and-plugin-errors branch March 3, 2026 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants