-
Notifications
You must be signed in to change notification settings - Fork 3
Update identify-web-user.mdx #188
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
base: main
Are you sure you want to change the base?
Conversation
no-work-item
🌟 EkLine ReviewerHello! I’m here to help improve your docs. I’ve reviewed your pull request, and left in-line suggestions for quick fixes. For details, visit the Analytics Page. For questions or feedback, please email [email protected]. |
Published docs preview URL: https://devrev-preview-270970c9-64c9-4c14-9285-ddb9c44bb9f3.docs.buildwithfern.com |
@@ -102,7 +102,7 @@ If you prefer a two-level structure, where users are directly associated with an | |||
|`account_ref` |A unique reference for the account. |string | | |||
|`display_name` | The name of the account that's shown on the widget. |string | | |||
|`domains` |The domain names of the accounts that the users belongs to. |array | | |||
| `phone_numbers` |The phone number associated with the account. |array | | |||
|`phone_numbers` |The phone number associated with the account must be in E.164 format. |array | |
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.
|`phone_numbers` |The phone number associated with the account must be in E.164 format. |array | | |
|`phone_numbers` |The phone number associated with the account in E.164 format. |array | |
Published docs preview URL: https://devrev-preview-e142812e-1b6b-4321-9c86-77fed3e83008.docs.buildwithfern.com |
Published docs preview URL: https://devrev-preview-25968cde-d85e-4a5d-ba72-940fd59a59cb.docs.buildwithfern.com |
Published docs preview URL: https://devrev-preview-51717e52-d810-495a-a058-83d3ee93a084.docs.buildwithfern.com |
Published docs preview URL: https://devrev-preview-4c40b426-9f4f-45c5-b18f-592b0ebfc041.docs.buildwithfern.com |
Published docs preview URL: https://devrev-preview-82455c2a-25ea-4c4f-b14d-87e84b60bf4b.docs.buildwithfern.com |
Published docs preview URL: https://devrev-preview-ebb2a6aa-0f08-41ee-bcf1-c5f9befc5178.docs.buildwithfern.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.
The suggestions are mostly superficial but please address them before merging.
|
||
DevRev offers three ways to structure your customer's data: | ||
1. **Single-level (Recommended for most B2C cases)** | ||
- User |
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.
Not sure what this means.
### How resolution works | ||
|
||
DevRev offers three ways to structure your customer's data: | ||
1. **Single-level (Recommended for most B2C cases)** |
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.
This list would probably be better as a table with columns number of levels, use case, user relationship, and required information.
If you prefer a two-level structure, where users are directly associated with an account instead of a workspace, you can provide the `user_ref` and details along with the `account_ref` and details. | ||
### API status codes and troubleshooting | ||
|
||
| Status code| Description & Troubleshooting | |
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.
Would be better to have troubleshooting at its own column.
| Status code| Description & Troubleshooting | | |
| Status code| Description | Troubleshooting | |
Co-authored-by: Ben Colborn <[email protected]>
Co-authored-by: Ben Colborn <[email protected]>
Co-authored-by: Ben Colborn <[email protected]>
Co-authored-by: Ben Colborn <[email protected]>
Published docs preview URL: https://devrev-preview-616fae00-346d-43a8-8f37-54ccfe5b2fc7.docs.buildwithfern.com |
Co-authored-by: Ben Colborn <[email protected]>
Published docs preview URL: https://devrev-preview-e7e43dc8-ec0a-48c4-969b-60e562b08a34.docs.buildwithfern.com |
Co-authored-by: Ben Colborn <[email protected]>
Published docs preview URL: https://devrev-preview-d86361f0-7006-46fe-a249-5571c1e772ff.docs.buildwithfern.com |
Published docs preview URL: https://devrev-preview-0a36aacd-a30e-40b6-a4c6-e7f5aebe6350.docs.buildwithfern.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.
Published docs preview URL: https://devrev-preview-7d1db411-8ff4-4266-a822-789d4355fe14.docs.buildwithfern.com |
Published docs preview URL: https://devrev-preview-469b347e-c1c7-4fe5-a93f-193a955ea1a7.docs.buildwithfern.com |
@Atul-Butola please either complete this PR or close it. |
@sambhavjain9138 , are we good to go live? This is open from a long time, please confirm. |
https://app.devrev.ai/devrev/works/ISS-162925