-
Notifications
You must be signed in to change notification settings - Fork 400
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
feat(tasks): track activity changes with document history. #5965
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
2a03eab
to
2880abf
Compare
Signed-off-by: Fred Carlsen <fred@sjelfull.no>
046858d
to
0ff5462
Compare
No changes to documentation |
Component Testing Report Updated Mar 12, 2024 4:52 PM (UTC)
|
0ff5462
to
e40e258
Compare
e40e258
to
0f3fdac
Compare
* @param fieldsToTrack - The fields to track for changes. | ||
* @returns An array of changes for the tracked fields. | ||
*/ | ||
export function trackFieldChanges( |
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 nice to have some tests added to this next week @sjelfull
|
||
const queryParams = `tag=sanity.studio.tasks.history&effectFormat=mendoza&excludeContent=true&includeIdentifiedDocumentsOnly=true&reverse=true` | ||
const transactionsUrl = client.getUrl( | ||
`/data/history/${dataset}/transactions/${task._id}?${queryParams}`, |
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.
is task._id
always guaranteed to be URL safe?
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.
It should always return a published id, so it should be safe, but I will modify it to force the use of a published id here, in case we enable the drafts later in the process.
Thanks
…warnings generated
9a020bd
to
a50eba7
Compare
* feat(tasks): track task changes with document history Signed-off-by: Fred Carlsen <fred@sjelfull.no> * feat(tasks): update tasks activity log to display values * fix(tasks): always fetch publishedId, add client api version * fix(tasks): update status selector and workspace provider to disable warnings generated --------- Signed-off-by: Fred Carlsen <fred@sjelfull.no> Co-authored-by: Fred Carlsen <fred@sjelfull.no>
Description
Adds tasks activity log.
Shows history changes made by users.
What to review
Testing
Notes for release