-
Notifications
You must be signed in to change notification settings - Fork 0
docs: add Billing Accounts capability and OAuth scopes (CXP-455) #37
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -19,9 +19,12 @@ sidebarTitle: "Coupa" | |||||||||||||
| | Groups | <Icon icon="square-check" iconType="solid" color="#c937ae"/> | <Icon icon="square-check" iconType="solid" color="#c937ae"/> | | ||||||||||||||
| | Roles | <Icon icon="square-check" iconType="solid" color="#c937ae"/> | <Icon icon="square-check" iconType="solid" color="#c937ae"/> | | ||||||||||||||
| | Licenses | <Icon icon="square-check" iconType="solid" color="#c937ae"/> | <Icon icon="square-check" iconType="solid" color="#c937ae"/> | | ||||||||||||||
| | Billing Accounts ² | <Icon icon="square-check" iconType="solid" color="#c937ae"/> | <Icon icon="square-check" iconType="solid" color="#c937ae"/> | | ||||||||||||||
|
|
||||||||||||||
| ¹ Account Groups sync is opt-in. To enable it, select **Account Groups** in the resource types to sync when configuring the connector in C1, and ensure the `core.accounting.read` OAuth scope is added to your Coupa OAuth client. | ||||||||||||||
|
|
||||||||||||||
| ² Billing Accounts sync requires the `core.accounting.read` OAuth scope. Provisioning (grant/revoke) additionally requires the `core.accounting.write` OAuth scope. | ||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The Account Groups footnote (¹) tells users that sync is opt-in and how to enable it. The Billing Accounts footnote (²) only mentions the OAuth scope requirement but is silent on whether it's opt-in or automatic. If Billing Accounts is also opt-in, consider adding similar guidance:
Suggested change
If it syncs automatically whenever the scope is present, the current wording works — but stating that explicitly would help distinguish it from Account Groups.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The Account Groups footnote (¹) explains that sync is opt-in and tells users how to enable it. This footnote only mentions scope requirements but doesn't clarify whether Billing Accounts sync is opt-in or automatic. If opt-in, consider adding similar guidance:
Suggested change
If it syncs automatically whenever the scope is present, stating that explicitly would help distinguish it from Account Groups. |
||||||||||||||
|
|
||||||||||||||
| ### Connector actions | ||||||||||||||
|
|
||||||||||||||
| Connector actions are custom capabilities that extend C1 automations with app-specific operations. You can use connector actions in the [Perform connector action](/product/admin/automations-steps-reference#perform-connector-action) automation step. | ||||||||||||||
|
|
@@ -77,7 +80,7 @@ A user with **Admin** access in Coupa must perform this task. | |||||||||||||
| - openid | ||||||||||||||
| - profile | ||||||||||||||
|
|
||||||||||||||
| **If you also want to sync Account Groups, add:** | ||||||||||||||
| **If you also want to sync Account Groups or Billing Accounts, add:** | ||||||||||||||
| - core.accounting.read | ||||||||||||||
|
|
||||||||||||||
| **You'll need these scopes to give C1 READ/WRITE access (syncing access data and provisioning access):** | ||||||||||||||
|
|
@@ -94,8 +97,9 @@ A user with **Admin** access in Coupa must perform this task. | |||||||||||||
| - openid | ||||||||||||||
| - profile | ||||||||||||||
|
|
||||||||||||||
| **If you also want to sync and provision Account Groups, add:** | ||||||||||||||
| **If you also want to sync and provision Account Groups or Billing Accounts, add:** | ||||||||||||||
| - core.accounting.read | ||||||||||||||
| - core.accounting.write (required for Billing Account provisioning) | ||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nit: "Billing Account" (singular) here vs "Billing Accounts" (plural) used everywhere else in this PR. Consider making it consistent:
Suggested change
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nit: "Billing Account" (singular) is inconsistent with "Billing Accounts" (plural) used everywhere else in this PR.
Suggested change
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nit: "Billing Account" (singular) is inconsistent with "Billing Accounts" (plural) used everywhere else in this PR.
Suggested change
|
||||||||||||||
| </Step> | ||||||||||||||
| <Step> | ||||||||||||||
| At the bottom of the page click **Save**. | ||||||||||||||
|
|
||||||||||||||
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 Account Groups footnote (¹) mentions that sync is opt-in and tells users to select the resource type in C1. Is Billing Accounts also opt-in? If so, it would be helpful to add similar guidance here (e.g., "To enable it, select Billing Accounts in the resource types to sync…"). If it syncs by default (whenever the scope is present), then the current wording is fine — but it might be worth stating that explicitly to distinguish it from Account Groups.