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

Add a link to OR’s CLI in the Automating section #391

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

antoine2711
Copy link
Member

Adding in Automating section a link to OR’s CLI from Felix Lohmeier.

Adding in automated section a link to OR’s CLI from Felix Lohmeier.
Copy link

netlify bot commented Oct 29, 2024

Deploy Preview for openrefine-website ready!

Name Link
🔨 Latest commit 24431e6
🔍 Latest deploy log https://app.netlify.com/sites/openrefine-website/deploys/67206edf6ac50d0008dab806
😎 Deploy Preview https://deploy-preview-391--openrefine-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@tfmorris
Copy link
Member

Wouldn't it be better to remove mentions of all these unsupported clients, since they could be considered implicit endorsements? They are using an undocumented, unsupported, and, most importantly, untested internal protocol which has very limited error handling/reporting (because it's designed to be used in a restricted web client with user supervision).

@thadguidry
Copy link
Member

@tfmorris I think we do a good job in the prior 2 paragraphs of explaining caveats, and "not responsible" attitudes well enough? No? But perhaps you'd like to suggest something additional into those paragraphs to make it more clear "not an endorsement", and "undocumented, untested, internal protocol used with limited error handling"?

@antoine2711
Copy link
Member Author

Wouldn't it be better to remove mentions of all these unsupported clients, since they could be considered implicit endorsements?

@tfmorris: I think they should go to a different page.

Personally, I would had another section, like “External tools and services”, and there, I would do a page for the clients (the section I’m modifying now), a subsection or a page for all the extentions, and a subsection/page for the Reconciliations Services.

They would all benefit from more explanation for the purpose, installation, usage, etc.

Regards, Antoine

@thadguidry
Copy link
Member

@antoine2711 Just throwing this out there, but... for things outside the control of OpenRefine, perhaps another alternative of where that information could be posted or captured into would be Wikibooks or Wikiversity articles?

@magdmartin and I had discussed places where the community might begin doing those sorts of things, like writing up things about OpenRefine and its ecosystem. Rather than us hosting a Wiki ourselves to allow the community to use, I felt that Wikibooks (textbooks or manuals) or Wikiversity (manuals or training resources, articles) might be better and save our team from maintenance pain?

@wetneb
Copy link
Member

wetneb commented Oct 30, 2024

In my opinion the existing warning is already pretty scary, I don't see a need to deter people from using those tools even more than what we currently do.

I also think we should re-evaluate this policy of discouraging people from using the HTTP API externally. The HTTP API is legitimately used by our own extensions - as seen with the issue created by removing CORS support from the get-rows command. It's difficult for me to understand why we'd decide to be so conservative for changes to Java classes (everything being part of the stable API, even the most obscure classes) and have no stability to offer on the frontend side.

@thadguidry
Copy link
Member

Ah, regarding the API, @wetneb that's actually a really good point and agree.

@magdmartin
Copy link
Member

magdmartin commented Oct 30, 2024

Should the documentation page redirect to our current listing on the main website: https://openrefine.org/extensions#client-libraries? This would avoid maintaining a list in two different places. We could also reproduce the warning on that page.

Those libraries are actively used. The preliminary results of the 2024 user survey show that about 7% of our users rely on one of those libraries (16 out of 226).

@thadguidry
Copy link
Member

thadguidry commented Oct 30, 2024

@magdmartin Agree, I'd suggest we move that small listing and caution under Automation. "Running" is just not semantically the same as "Using it with...(i.e. automating, etc.)". That would leave the Automation section on running.md to be very lightweight paragraph where we can then provide a link to the extensions.md clients-libraries section.

All in favor?

@antoine2711
Copy link
Member Author

antoine2711 commented Oct 30, 2024

This would avoid maintaining a list in two different places.

@magdmartin : why should the documentation about tools, extensions and services be out of our current documentation? What’s the advantage for a user to have to look at 2 different places? That why I’m proposing to had a third section in our documentation. Here’s the link to the forum discussion about that: Improving the documentation with a new section, “External tools and Services”

Regards, Antoine

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.

5 participants