Skip to content

Add lastModifiedDate from version to the object response#301

Merged
TimCsaky merged 2 commits intomasterfrom
SC3864
Apr 22, 2025
Merged

Add lastModifiedDate from version to the object response#301
TimCsaky merged 2 commits intomasterfrom
SC3864

Conversation

@jatindersingh93
Copy link
Contributor

@jatindersingh93 jatindersingh93 commented Apr 14, 2025

  • Adding 'lastModifiedDate' to the search result, which can replace 'updated date' in the object list view.

Description

Types of changes

Checklist

  • I have read the CONTRIBUTING doc
  • I have checked that unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Further comments

@github-actions
Copy link

github-actions bot commented Apr 14, 2025

Coverage Report

Totals Coverage
Statements: 57.08% ( 3066 / 5371 )
Methods: 47.11% ( 334 / 709 )
Lines: 63.87% ( 1842 / 2884 )
Branches: 50.06% ( 890 / 1778 )

@jatindersingh93 jatindersingh93 marked this pull request as ready for review April 16, 2025 23:37
@TimCsaky
Copy link
Contributor

we can also include lastodifiedDate in the response from createObject and updateObject.
in controllers/object.js.. for updateObjecthere and for createObject here add this line:
object.lastModifiedDate = version.lastModifiedDate

@norrisng-bc
Copy link
Contributor

updatedAt:
type: string
format: date-time
description: Time when this record was last updated
example: "2022-03-11T23:19:16.343Z"

I'd probably also clarify in the description that it's referring to a COMS record - specifically, operations done via COMS as well as when an external change was picked up during a sync.

@TimCsaky TimCsaky merged commit fc0f8e8 into master Apr 22, 2025
13 checks passed
jatindersingh93 pushed a commit that referenced this pull request Apr 28, 2025
Add lastModifiedDate from version to the object response
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants