Skip to content

Feature/DF-861: Geospatial component#321

Merged
davidjamesstone merged 72 commits intomainfrom
feature/DF-861-geospatial-component
Mar 25, 2026
Merged

Feature/DF-861: Geospatial component#321
davidjamesstone merged 72 commits intomainfrom
feature/DF-861-geospatial-component

Conversation

@davidjamesstone
Copy link
Contributor

@davidjamesstone davidjamesstone commented Feb 27, 2026

Proposed change

Jira ticket: https://eaflood.atlassian.net/browse/DF-861

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Misc. (documentation, build updates, etc)

Checklist

  • You have executed this code locally and it performs as expected.
  • You have added tests to verify your code works.
  • You have added code comments and JSDoc, where appropriate.
  • There is no commented-out code.
  • You have added developer docs in README.md and docs/* (where appropriate, e.g. new features).
  • The tests are passing (npm run test).
  • The linting checks are passing (npm run lint).
  • The code has been formatted (npm run format).

@davidjamesstone davidjamesstone marked this pull request as ready for review March 19, 2026 10:21
Copy link
Contributor

@jbarnsley10 jbarnsley10 left a comment

Choose a reason for hiding this comment

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

Really excellent work! A few comments but nothing major. I'd like to see the test coverage once the pipeline is working as may be difficult to cover map.js, geospatial-map.js etc

Copy link
Contributor

@jbarnsley10 jbarnsley10 left a comment

Choose a reason for hiding this comment

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

Nice one!

@sonarqubecloud
Copy link

@davidjamesstone davidjamesstone merged commit a71a8c5 into main Mar 25, 2026
23 checks passed
@davidjamesstone davidjamesstone deleted the feature/DF-861-geospatial-component branch March 25, 2026 09:59
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