-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Feat: GA Change Tracking feature #21697
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: develop
Are you sure you want to change the base?
Conversation
Hi @WriteMayur 👋 Thanks for your pull request! Your PR is in a queue, and a writer will take a look soon. We generally publish small edits within one business day, and larger edits within three days. Please ensure the propsed changes look good by building it first in your local environment. Refer to this contribution guide to get the site up and running in your local. If you really require a preview url, reach out to one of the writers and they will generate one for you. |
netlify build |
🚀 Netlify Preview Building!If the build is successful, the preview for this pull request will be available at the following URL (usually takes 10-20 minutes): |
netlify build |
netlify build |
netlify build |
netlify build |
netlify build |
netlify build |
* **Flexible integration**: Configure via APIs, CLI, or CI/CD pipeline integrations such as GitHub Actions and Jenkins | ||
|
||
<Video | ||
id="ikRUKJf7hyU" |
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 video is three years old and is quite outdated. The UI and functionality has changed significantly since then.
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.
@nr-jonathanpdx, I have temporarily commented out the existing video. Once we receive a new or updated link, I will be sure to add it back in.
id="optional-mutation-example" | ||
title="Mutation with optional fields" | ||
> | ||
```graphql |
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.
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.
@nr-jonathanpdx, I've raised this issue with our Docs Engineering team in this slack thread. I'll let you know as soon as we receive an update from them.
|
||
* Time range selector for filtering events | ||
* Table sorting and filtering options | ||
* Up to 2,000 changes loaded (adjust time range if needed) |
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.
I believe this should be 1,000 changes.
</tr> | ||
<tr> | ||
<td>Change tracking tables</td> | ||
<td>2,000 events loaded by default</td> |
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.
As above, I think this is 1,000?
</tr> | ||
<tr> | ||
<td>Change tracking tables</td> | ||
<td>2,000 events loaded by default</td> |
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.
As elsewhere, I believe this number should be 1,000
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.
@kaacarroll, Thanks for the feedback! Could you please confirm that 1,000 is the correct value before I make the update?
netlify build |
netlify build |
This PR restructure and updates the change tracking documentation for its GA release.