Skip to content

feat: add modified date timestamps to artifacts#205

Closed
gasperzgonec wants to merge 7 commits into
mainfrom
gasperz/ASCPT-22
Closed

feat: add modified date timestamps to artifacts#205
gasperzgonec wants to merge 7 commits into
mainfrom
gasperz/ASCPT-22

Conversation

@gasperzgonec

@gasperzgonec gasperzgonec commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Description

This change adds timestamps (number of ms since EPOCH) to artifacts, to improve debuggability and improve the reporting by adding user-readable time information of last extraction.

Before

      {
        id: 'artifact-1780473180514-6y1t76ilf',
        item_type: 'todos',
        item_count: 15
      },

New

      {
        id: 'artifact-1780476433181-f5fvke1ps',
        item_type: 'todos',
        item_count: 15,
        oldest_created_date: 1527840000000,
        newest_created_date: 1768035600000,
        oldest_modified_date: 1527840000000,
        newest_modified_date: 1772359200000
      },

Connected Issues

Checklist

  • Tests added/updated and ran with npm run test OR no tests needed.
  • Ran backwards compatibility tests with npm run test:backwards-compatibility.
  • Code formatted and checked with npm run lint.
  • Tested airdrop-template linked to this PR.
  • Documentation updated and provided a link to PR / new docs OR no docs needed.

@gasperzgonec gasperzgonec requested review from a team and radovanjorgic as code owners June 3, 2026 07:59
@gasperzgonec gasperzgonec requested a review from GasperSenk June 3, 2026 08:00
Comment thread src/uploader/uploader.helpers.ts Outdated
Comment thread src/uploader/uploader.helpers.ts Outdated
Comment thread src/uploader/uploader.ts Outdated
@radovanjorgic radovanjorgic marked this pull request as draft June 9, 2026 09:57
@radovanjorgic radovanjorgic changed the title feat: Add timestamps to artifacts feat: add modified date timestamps to artifacts Jun 11, 2026
@radovanjorgic

Copy link
Copy Markdown
Collaborator

Closing because we opened new PR which merges #202 and #205

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