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

OCS Resolve API issues (missing ID, verbose response) #1555

Open
stefan-niedermann opened this issue Jan 25, 2025 · 1 comment
Open

OCS Resolve API issues (missing ID, verbose response) #1555

stefan-niedermann opened this issue Jan 25, 2025 · 1 comment
Labels
0. Needs triage Pending approval or rejection. This issue is pending approval. enhancement New feature or request

Comments

@stefan-niedermann
Copy link
Member

Is your feature request related to a problem? Please describe.

The OCS Resolve endoint (GET https://example.com/ocs/v2.php/references/resolve?reference=https%3A%2F%2Fexample.com%2Findex.php%2Fapps%2Ftables%2F%23%2Ftable%2F10) currently returns a richObject property, that doesn't actually satisfy me.

First of all, it includes all data from that table. Given this endpoint gets hit when a user searches for anything, that seems to be a bit too verbose to me.

Also: I expected (and require) the most obvious information: the ID of the object. Sure, I could parse it from the URL, but what if the UI changes at some point in the time? No, I'd like the ID to be present in the response if possible 🙂

Describe the solution you'd like

Drop the rows property, the summarizing row count is enough, add the id property.

Describe alternatives you've considered

No response

Additional context

No response

@stefan-niedermann stefan-niedermann added 0. Needs triage Pending approval or rejection. This issue is pending approval. enhancement New feature or request labels Jan 25, 2025
@blizzz
Copy link
Member

blizzz commented Jan 29, 2025

Rows were added in #426

@juliusknorr do you recall the details? I do not see a related issue in the list. The commit CLOB does not have pointers either.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending approval or rejection. This issue is pending approval. enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants