Laurence Isla
Laurence Isla
> @laurenceisla Since you've been working on OpenAPI, do you know if there's a way to represent this metadata? (whether openAPI v2 or v3) One way would be to include...
I think this is a special case of #1130. Should we close this issue and continue there?
To give a similar example with our tests: ```bash curl -X PATCH 'localhost:3000/entities?select=id,name&or=(name.is.null,name.like.*test*)' \ -H 'Content-Type: application/json' \ -H 'Prefer: return=representation' \ -d '{"name": "New name"}' ``` Got: ```json []...
> The only thing is that postgrest-openapi is not usable yet AFAICT Yes, it's not completely ported. The most important missing features are functions support, permissions and FK relationships (the...
Related issue: https://github.com/PostgREST/postgrest/issues/1840 > I.e. give them our own PGRST error code. Agree.
My approach right now is to generate this query for a to-many request: ```bash curl 'localhost:3000/clients?select=name,...projects(name,id)' ``` ```sql SELECT "test"."clients"."name", "clients_projects_1"."name", "clients_projects_1"."id" FROM "test"."clients" LEFT JOIN LATERAL ( SELECT json_agg("projects_1"."name")...
Some caveats I encountered: ### Repeated values and order Do we want to keep repeated values in the results? For example (not the best use case, just to illustrate): ```bash...
OK, this is what I got implemented so far. For example, using the tables in our spec test: ``` Factories processes supervisors ``` ```bash curl 'localhost:3000/factories?select=name,...processes(processes:name,...supervisors(supervisors:name))' ``` ```javascript [ {...
> This has happened somewhere outside this PR on main though. Found the loadtest in which `head` and `main` started to differ: https://github.com/PostgREST/postgrest/actions/runs/9846138600/job/27183597190 After some local tests, I think the...
> So the regression really only happens on the root endpoint for the openapi output. Given that we're working towards removing / replacing that anyway, I don't consider this regression...