Skip to content
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

Improve React Native SDK description #326

Merged
merged 4 commits into from
Sep 30, 2024

Conversation

andychow326
Copy link
Contributor

ref DEV-2040

@louischan-oursky
Copy link
Collaborator

@louischan-oursky

* Enable biometric authentication for the signed in user.
* Platform specific biometric authentication UI will be shown.
* You may first enable biometric authentication in Authgear Portal (Your Project \> Authentication \> Biometric \> Enable biometric authentication).
* Otherwise, `BiometricDisallowed` error will be thrown.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We do not have BiometricDisallowed error. Remove this line.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

* Authenticate with biometric authentication.
* Platform specific biometric authentication UI will be shown.
* You may first enable biometric authentication in Authgear Portal (Your Project \> Authentication \> Biometric \> Enable biometric authentication).
* Otherwise, `BiometricDisallowed` error will be thrown.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We do not have BiometricDisallowed error. Remove this line.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

/*
* The URL to open.
/**
* The target authorization URL.
*/
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change the description to The URL to be opened by the UIImplementation.

This documentation is for UIImplementation implementor. So from their point of view, they want to know how to implement a UIImplementation. The documentation should guide them.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

* The URL to detect.
/**
* The value should be a valid Redirect URI to which the response will be sent after authentication.
* You must also add a Redirect URI in Authgear Poral via the Redirect URI section of your Authgear Application.
*/
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change the description to The URL to be detected by the UIImplementation. When this URL is detected, the UIImplementation MUST return this URL, and close itself.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

* A flag to some implementations that can share cookies with the device browser.
* When it is set to true, cookies will be shared with the device browser.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change the description to A flag to tell the UIImplementation that cookies should be shared with the device browser. This flag is only useful to UIImplementation that can share cookies with the device browser, such as those underlying implementations are based ASWebAuthenticationSession, or CustomTabs.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@andychow326 andychow326 force-pushed the 2040-react-native-docs branch from 07b63cf to f624786 Compare September 27, 2024 09:25
@andychow326
Copy link
Contributor Author

@louischan-oursky Updated! :adore2x:

@louischan-oursky louischan-oursky merged commit f77c1f5 into authgear:master Sep 30, 2024
9 checks passed
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