386 datasetlastmodifieddate is not updated

clowder-framework / clowder

A data management system that allows users to share, annotate, organize and analyze large collections of datasets. It provides support for extensible metadata annotation using JSON-LD and a distribute analytics event bus for automatic curation of uploaded data.

https://clowderframework.org/

University of Illinois/NCSA Open Source License

34 stars 17 forks source link

386 datasetlastmodifieddate is not updated #392

Closed ddey2 closed 1 year ago

ddey2 commented 1 year ago

Description

Review Time Estimate

[ ] Immediately
[ ] Within one week
[x] When possible

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My change requires a change to the documentation.
[x] I have updated the CHANGELOG.md.
[x] I have signed the CLA
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

tcnichol commented 1 year ago

I pushed a small change. I was getting some errors but they were small fixes.

I have tried adding files and deleting them, but it doesn't seem my dataset is showing the lastUpdated time changing. Will delete this db and start over and try again.

ddey2 commented 1 year ago

@tcnichol Thanks, I am looking into this.

tcnichol commented 1 year ago

I made one commit - in datasets.submit i added the same update of the time you had elsewhere. I think that catches it when files are uploaded through the web interface and not the API and will now update the last modified time.

ddey2 commented 1 year ago

Adding last modified date to UI