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

API specification (v2) does not document fullTextEnabled #102

Open
1 task done
JKingweb opened this issue Feb 26, 2017 · 9 comments
Open
1 task done

API specification (v2) does not document fullTextEnabled #102

JKingweb opened this issue Feb 26, 2017 · 9 comments
Labels
1. to develop API Impact API/Backend code enhancement

Comments

@JKingweb
Copy link

JKingweb commented Feb 26, 2017

IMPORTANT

Read and tick the following checkbox after you have created the issue or place an x inside the brackets ;)

Explain the Problem

The API specification for the v2 API does not document what the 'fullTextEnabled' feed property means. The method docs say only "see Feeds", but the Feeds section does not document the property.

I gather (after reading CONTRIBUTING :) ) that this is about PicoFeed full-content scraping rather than full-text searching as I had first assumed, but even knowing this it's not clear how it should/can be used.


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@BernhardPosselt
Copy link
Member

Can you create a pull request in that case? It's still a draft BTW nothing is implemented yet

@JKingweb
Copy link
Author

As I'm not sure what the spec should be saying on the subject, I don't think it would make sense for me to create a pull request.

The 10.0.0 changelog suggests a new update API is now in use. I took that to mean the v2 API had been implemented; is that not the case, then? I've not looked at the software yet; I've just been reading the specs.

@BernhardPosselt
Copy link
Member

Yeah, misunderstanding :) that one was about cron job updates which had to be done because the old APIs were removed.

There's an api call that lists available APIs which you can query.

The full text enabled basically turns on full text feeds from picofeed as in: the target website still be parsed and not the RSS entry

@Thaodan
Copy link
Contributor

Thaodan commented Nov 18, 2022

The full text enabled basically turns on full text feeds from picofeed as in: the target website still be parsed and not the RSS entry

I think it makes sense to mention that some feeds already have the full contents of the website for that entry.
In such cases there should be some feedback for the user.

Does this also affects clients that fetch feeds from the nextcloud app?

@Grotax
Copy link
Member

Grotax commented Nov 18, 2022

API v2 is not even implemented yet.

But yes it would affect clients.

The api is currently only used by clients the web interface is not really using the api but other ways.

@Thaodan
Copy link
Contributor

Thaodan commented Nov 18, 2022 via email

@SMillerDev
Copy link
Contributor

What does that mean? The web interface shows an option to enable this feature, this must mean that it is using Apiv2.

No, quite the opposite. This issue says that there is no V2 api to enable full text.

@Thaodan
Copy link
Contributor

Thaodan commented Nov 18, 2022 via email

@Grotax
Copy link
Member

Grotax commented Nov 19, 2022

What I meant in my first comment was that the integrated web interface is not using the API that news provides to other clients.

Therefore functionality and features are different.

The webclient has the option to enable full text but it doesn't do that over a public API endpoint.

I hope that clears up some confusion.

External apps currently use API v1.2 or v1.3 which brought a few new options.
v2 was only drafted years ago but never fully implemented.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop API Impact API/Backend code enhancement
Projects
None yet
Development

No branches or pull requests

5 participants