-
Notifications
You must be signed in to change notification settings - Fork 2
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Update sdk to match authsignal's general sdk conventions (#24)
* Change apiBase to apiURL * Add updateAction function * Make error handling consistent with general Authsignal SDK conventions * Update track to use named params within array * Use single associative array parameter instead of positional params * Remove redirectUrl from getUser function * Replace data with attributes * Add getAuthenticators method * Fix tests * Add updateAction test * Add PHPDoc comment to Update User * Change order of functions to match Node SDK * Change updateUser to use patch instead of post * Reorder tests to match order of SDK functions * Make attributes optional in track request * Replace authenticator with attributes param * Update AuthsignalTest.php * Replace apiKey with apiSecretKey * Rename setApiHostname to setApiUrl * Remove `setApiVersion` and `getApiVersion`. Rename env var AUTHSIGNAL_SERVER_API_ENDPOINT to AUTHSIGNAL_API_URL * Fix tests
- Loading branch information
1 parent
877abd6
commit 108a2ac
Showing
6 changed files
with
365 additions
and
275 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.