-
Notifications
You must be signed in to change notification settings - Fork 2
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
Make 'projection' extension mandatory #26
Comments
This doesn't seem to work on Geostore non-prod. This the collection.json I was using for testing |
Can't seem to attach json files anymore, here it is.
|
If I understand the report correctly, this is because we don't currently support the latest version of the "linz" LINZ STAC extension. This is because we don't automatically update the supported version whenever a new extension is released. This would be difficult to automate, so I would suggest creating a Geostore issue whenever we want the Geostore to use a new reference extension version. The currently supported version is in backend/stac_format.py. |
Hey @MitchellPaff @l0b0. This doesn't seem to be working. If I don't have Tested on nonprod today. |
@billgeo I think there's a misunderstanding here - the extensions repo isn't part of the Geostore. For a change in the extensions repo to appear in the Geostore we need to release a new version of the extensions and then bump the version of the extensions in the Geostore repo. If we need to incorporate a change into the Geostore that should either be mentioned in the issue or be created as a separate issue in the Geostore, dependent on the issue in the STAC extensions repo. |
Ah right. yes. That happened last time didn't it. Is it possible bump the version in nonprod (but not prod), so I can test it whenever we work on a story like this? Otherwise I have no way to test it. Would we need to add this to the definition of done? |
You can test it in this repository - there shouldn't be any need to test it in the Geostore every time. |
Found a acouple of things while testing...
|
This looks good. I've started watching the stac-extension issue so will pick that up when we get an answer to that. |
1 similar comment
This looks good. I've started watching the stac-extension issue so will pick that up when we get an answer to that. |
In the LINZ extension make the 'projection' extension mandatory.
The text was updated successfully, but these errors were encountered: