Skip to content

Conversation

@lubianat
Copy link
Contributor

@lubianat lubianat commented Dec 2, 2025

Current README mixes .rst and .md syntax, and GitHub fails to render links correctly

grafik

README.rst Outdated
Implementation of [OME-NGFF](https://ngff.openmicroscopy.org/latest/) API with an omero-web app.
Implementation of `OME-NGFF <https://ngff.openmicroscopy.org/latest/>`_ API with an omero-web app.

This plugin supports OME-NGFF v0.1, v03, v0.4.
Copy link
Member

Choose a reason for hiding this comment

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

Could you also remove the v0.1 here as I think that is wrong.

And in the line below, could you update it like the example URLs at the bottom.
e.g.

https://[omero-server]/zarr/v0.4/image/[ID].zarr/

Great, thanks

@will-moore
Copy link
Member

Thanks for the PR. https://github.com/lubianat/omero-web-zarr/tree/patch-1 README looks good.
I just noticed a couple of other issues while I was reading (see above).
Would be nice to get them fixed in this PR but not a blocker...

@lubianat
Copy link
Contributor Author

lubianat commented Dec 2, 2025

@will-moore like this? feel free to hijack the PR to indicate or add more doc changes (maintainers should have push access to the branch, I think)

@will-moore will-moore merged commit 4e1e685 into ome:main Dec 2, 2025
1 check passed
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