Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove DEA Waterbodies v1 from DEA Maps and old raster V0 from repo #1230

Merged
merged 3 commits into from
Feb 29, 2024

Conversation

BexDunn
Copy link
Contributor

@BexDunn BexDunn commented Feb 29, 2024

This PR is based off an approved CCB request.

This PR is to remove the listing on DEA Maps for DEA Waterbodies v1, which is no longer produced, and to remove the indexing of the v0 DEA Waterbodies raster (which is from pre-2019 and was missed in a previous deletion effort). v1 would remain on terria maps for any majorly impacted stakeholders and v2 remains on both terria and DEA maps.

Old yaml https://github.com/GeoscienceAustralia/dea-config/pull/1230/files#diff-a669c4855953c4f9dd71fa49e8126d2a3ff1499786b6b6f8fa9ff3878c6ceffd was the source for this, which is very very obsolete.
https://explorer.sandbox.dea.ga.gov.au/products/water_bodies

@BexDunn BexDunn changed the title Wb v1 remove Remove DEA Waterbodies v1 from DEA Maps and old raster V0 from repo Feb 29, 2024
@BexDunn BexDunn requested review from robbibt and omad February 29, 2024 04:41
@BexDunn BexDunn marked this pull request as ready for review February 29, 2024 04:46
Copy link
Member

@robbibt robbibt left a comment

Choose a reason for hiding this comment

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

Looks good to me! Have checked the preview bot link and DEA Maps is still working nicely

@BexDunn BexDunn merged commit f0a3bff into master Feb 29, 2024
6 checks passed
@BexDunn BexDunn deleted the wb_v1_remove branch February 29, 2024 05:12
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