Skip to content
This repository has been archived by the owner on Oct 4, 2021. It is now read-only.

Add multi-access support to get, patch, and remove #6

Open
Sieabah opened this issue Sep 6, 2018 · 0 comments
Open

Add multi-access support to get, patch, and remove #6

Sieabah opened this issue Sep 6, 2018 · 0 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@Sieabah
Copy link
Owner

Sieabah commented Sep 6, 2018

Feathers Crow updated access patterns for default behavior. Those have been implemented, however support for those features lacks in three service methods.

Get should be supported instead of relying on just find
Patch should be supported, becomes an issue to test as couchbase doesn't support N1QL during mocking
Remove should be supported, same issues as patch.

There are also other issues with patch & remove when fetching data as couchbase doesn't guarantee any order, optionally we could sort by created_at or updated_at?

@Sieabah Sieabah added enhancement New feature or request help wanted Extra attention is needed labels Sep 6, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant