Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

fix: @feathersjs/adapter-commons: update id is non-nullable#1468

Merged
daffl merged 1 commit intofeathersjs:masterfeathersjs/feathers:masterfrom
vonagam:fix-update-id-typevonagam/feathers:fix-update-id-typeCopy head branch name to clipboard
Jul 19, 2019
Merged

fix: @feathersjs/adapter-commons: update id is non-nullable#1468
daffl merged 1 commit intofeathersjs:masterfeathersjs/feathers:masterfrom
vonagam:fix-update-id-typevonagam/feathers:fix-update-id-typeCopy head branch name to clipboard

Conversation

@vonagam
Copy link
Member

@vonagam vonagam commented Jul 18, 2019

Reflect in id type that update method never supports multi-record changes.

@daffl
Copy link
Member

daffl commented Jul 19, 2019

Thank you!

@daffl daffl merged commit 43ec802 into feathersjs:master Jul 19, 2019
@vonagam vonagam deleted the fix-update-id-type branch September 5, 2019 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.