Feature: add support for `service.id` as array of columns
See original GitHub issueWhen integrating this lib with feathers-objection & feathers-cassandra services, service.id
property can be an array of column names that represent a composite PK.
In addition, there’s the service.idSeperator
property that default to ,
so a composite PK can be set as ID with REST, e.g. /users/firstPathParam,secondPathParam
.
I didn’t find any workaround for this issue in the lib options.
The only thing that worked was changing the file feathers-swagger/openapi.js:line 190
to:
swaggerPath += `/{${Array.isArray(methodIdName) ? methodIdName.join(`}${service.idSeparator}{`) : methodIdName}}`;
Is there anything I’m missing here? should we integrate this logic into the lib or expose a method in the options to control it?
Issue Analytics
- State:
- Created 3 years ago
- Comments:6 (3 by maintainers)
Top Results From Across the Web
Work with arrays in Google Standard SQL | Cloud Spanner
With Google Standard SQL, you can construct array literals, build arrays from subqueries using the ARRAY function, and aggregate values into an array...
Read more >Query (Feature Service/Layer)—ArcGIS REST APIs
The query operation returns either a feature set, an array of feature IDs, and/or a result extent.
Read more >Using arrays in Google Sheets - Google Docs Editors Help
An array is a table (consisting of rows and columns) of values. If you want to group the values of your cells together...
Read more >array_column - Manual - PHP
array_column () returns the values from a single column of the array , identified by the column_key . Optionally, an index_key may be...
Read more >ARRAY | CockroachDB Docs
CockroachDB supports indexing array columns with GIN indexes. ... Adding the term ARRAY to any non-array data type. ... Using the array_append function....
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
PRs are welcome. As a hint, if you need it fast, you should target v1 branch, I guess master (and v2) will still take a while. Of course this would only be valid if there are no BC breaks.
Merged it into v1 branch.