Add BAIS2 UDP entry for Development Seed#316
Add BAIS2 UDP entry for Development Seed#316zacdezgeo wants to merge 9 commits intoESA-APEx:mainfrom
Conversation
Create provider record, add BAIS2 service record and UDP JSON, and include preview/thumbnail images from the notebook output.
|
Thank you @zacdezgeo for this initial PR and setting up the records. My apologies for the many comments, but it seems that the records are still requiring some information to pass the unit test. I see that our documentation is missing a part where you can execute the unit tests locally, but I did this by following these steps:
I would expect the unit tests to be running from the PR automatically, but it seems that they are not getting started... Unfortunately, there is also a known bug where the preview function of the catalogue is not working from remote forks. However, I suggest that once the records have been updated, we duplicate the branch to this repository, allowing you to visually inspect how the record would look like in the catalogue. |
Add TiTiler openEO platform record and link BAIS2
|
Thank you so much for the detailed review and for taking the time to leave all of these comments. I really appreciate it. This is my first contribution to the I’ll work through the requested updates and rerun the QA validation locally with Thanks again for the support in navigating the contribution process here, it’s very appreciated. |
|
I followed up on your comments regarding the record's links, but I still need to add the platform's logo (dark and light) based on the results from running: |
|
Thank you very much for the updates @zacdezgeo and my apologies for the delayed review. Everything looks good now, if you can just update the branch to be up-to-date with the latest changes, I can merge it to the main branch so that you can see the resulting service. |
|
@JanssenBrm ; just updated! let me know if there's any next steps i can look into |
|
Thank you @zacdezgeo, for the last updates. As the preview feature is not working from a remote fork, due to privileges, I have merged your changes into a new PR: #378. I'm going to close this one so that we can finalize everything in the other PR. |
Create provider record, add BAIS2 service record and UDP JSON, and include preview/thumbnail images from the notebook output.