Improvements to Adapter: unlinkAccount and updateUser #11813
bjoernwuest
started this conversation in
Ideas
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Goals
Simplify / streamline Adapter API.
Non-Goals
Introduce new functionality
Background
In current Adapter interface, following functions are defined which deviate from all other function specifications:
unlinkAccount: Promise<undefined | AdapterAccount>
... as only function, this returns "undefined" whereas other functions return null.updateUser: Promise<AdapterUser>
... this function must return a user, even no user to update was found!Proposal
Update functions to following syntax:
unlinkAccount(...): Promise<null | AdapterAccount>
updateUser(...): Promise<null | AdapterUser>
Beta Was this translation helpful? Give feedback.
All reactions