-
Notifications
You must be signed in to change notification settings - Fork 41
Update advanced sync settings documentation #1756
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
Clarified sync settings details and added examples for retrieving and updating sync settings.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Corrected spelling and improved clarity regarding data synchronization settings.
| --- | ||
|
|
||
| For most of our data types, we retrieve all available history. For financial statement data types (`balanceSheet`, `profitAndLoss`, `cashFlowStatement`), we retrieve 24 months of history. You can apply additional sync settings to change these defaults. This helps you limit the amount of data synchronized from the source platform. | ||
| For most of our data types, we retrieve all available history. For financial statement data types, (`balanceSheet`, `profitAndLoss`, `cashFlowStatement`), we retrieve 24 months of history. |
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.
📝 [vale] reported by reviewdog 🐶
[Google.Parens] Use parentheses judiciously.
| - When using advanced sync settings, settle on a uniform approach across your data types (e.g. all data types request 24 months' data). Some data types share requests, and applying different approaches may cause inconsistency. | ||
|
|
||
| - Advanced sync settings can be applied to all data types, but we advise you do not set them for reference data, such as customers, suppliers, chart of accounts, tax rates, tracking categories, and items. | ||
| - Advanced sync settings can be applied to all data types, but we advise you don't set them for reference data, such as customers, suppliers, chart of accounts, tax rates, tracking categories, and items. |
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.
📝 [vale] reported by reviewdog 🐶
[Google.Passive] In general, use active voice instead of passive voice ('be applied').
| GET /profile/syncSettings | ||
| ``` | ||
|
|
||
| To override client-level sync settings at a company level, use the [Update company sync settings](/platform-api#/operations/get-profile-syncSettings) endpoint: |
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.
@pj-simpson as discussed, URL needs to be amended once the endpoint is back in the OAS
| ``` | ||
|
|
||
| For example, to configure advanced settings for our `invoices` data type, send the following request to the endpoint: | ||
| For example, to configure company-specific settings for the `invoices` data type, send the following request to the endpoint: |
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.
@pj-simpson the example code needs to change to be company-specific
|
Link check results for preview deployment (https://codat-docs-git-advanced-sync-settings-codat.vercel.app): |
In support we are often sending the 'Advanced Sync Settings' article, however it explicitly referenced 'Delta/ Full syncs' which are an implementation detail that we do not wish t expose to our consumers going forward.
The article also contained a link to dead API docs.
When necessary, when advising our clients about Advanced Sync Settings, we want to be able to provide them with a 'one shot' article which they can read and go away with clear next steps in mind.
Have added
httpexamples to the markdown code.