New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
passkey config admin changes #2326
base: master
Are you sure you want to change the base?
Conversation
Co-authored-by: Kevin Cheung <kevinthecheung@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
what is the plan for integration tests? |
@Xiaoshouzi-gh integration tests have been drafted in this PR. Waiting on backend to allow this. |
@@ -69,4 +72,13 @@ export class Auth extends BaseAuth { | |||
public projectConfigManager(): ProjectConfigManager { | |||
return this.projectConfigManager_; | |||
} | |||
|
|||
/** | |||
* Returns the passkey config manager instance. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you update this api reference? looks redundant.
src/auth/passkey-config.ts
Outdated
); | ||
} | ||
// Validation for updating an existing PasskeyConfig. | ||
if (!isCreateRequest && typeof rpId !== 'undefined') { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you allow customer to pass in the existing value rpId for updates?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't that seem a little redundant? Since we can't change rpId once created?
* user record changes for getAccountInfo() * lint and api-extractor fixes * Apply suggestions from code review Co-authored-by: Kevin Cheung <kevinthecheung@users.noreply.github.com> * remove `[key: string]: unknown;` field from `PasskeyInfoResponse` * add undefined displayName case * name and credentialId are not optional * add `rpId` to update --------- Co-authored-by: Kevin Cheung <kevinthecheung@users.noreply.github.com>
No description provided.