[PR #1207] [CLOSED] Add config for using timestamp as Snapshot.added #4367

New issue

Closed

opened 2026-03-15 01:40:40 +03:00 by kerem · 0 comments

kerem commented

2026-03-15 01:40:40 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/ArchiveBox/ArchiveBox/pull/1207
Author: @overhacked
Created: 8/8/2023
Status: ❌ Closed

Base: dev ← Head: timestamp_as_added

📝 Commits (1)

0892645 Add config for using timestamp as Snapshot.added

📊 Changes

4 files changed (+31 additions, -1 deletions)

View changed files

📝 archivebox/config.py (+1 -0)
➕ archivebox/core/migrations/0022_auto_20230320_1814.py (+19 -0)
📝 archivebox/core/models.py (+2 -1)
📝 archivebox/index/sql.py (+9 -0)

📄 Description

Summary

💥 IMPORTANT: This PR includes a Django migration. 💥

Adds ability to set added field to the imported timestamp. Useful when importing from RSS or other rich formats like JSON that have past timestamps in the import metadata, so the ArchiveBox interface can display the source imported times instead of a large batch all with the same added date.

Open questions before merging

Should this behavior be default turned off?
Should it be configurable at all?
Should it be turned on, but only when importing from a metadata-rich parser?

Fixes #785.

Changes these areas

Bugfixes
Feature behavior
Command line interface
Configuration options
Internal architecture
Snapshot data layout on disk

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/ArchiveBox/ArchiveBox/pull/1207 **Author:** [@overhacked](https://github.com/overhacked) **Created:** 8/8/2023 **Status:** ❌ Closed **Base:** `dev` ← **Head:** `timestamp_as_added` --- ### 📝 Commits (1) - [`0892645`](https://github.com/ArchiveBox/ArchiveBox/commit/0892645c323777bb80ea4e312a1a93a8342ccaf2) Add config for using timestamp as Snapshot.added ### 📊 Changes **4 files changed** (+31 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `archivebox/config.py` (+1 -0) ➕ `archivebox/core/migrations/0022_auto_20230320_1814.py` (+19 -0) 📝 `archivebox/core/models.py` (+2 -1) 📝 `archivebox/index/sql.py` (+9 -0) </details> ### 📄 Description  # Summary :boom: **IMPORTANT: This PR includes a Django migration.** :boom: Adds ability to set `added` field to the imported `timestamp`. Useful when importing from RSS or other rich formats like JSON that have past timestamps in the import metadata, so the ArchiveBox interface can display the source imported times instead of a large batch all with the same added date. ## Open questions before merging - **Should this behavior be default turned off?** - Should it be configurable at all? - Should it be turned on, but only when importing from a metadata-rich parser?  # Related issues Fixes #785.  # Changes these areas - [x] Bugfixes - [x] Feature behavior - [ ] Command line interface - [ ] Configuration options - [ ] Internal architecture - [ ] Snapshot data layout on disk --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-03-15 01:40:40 +03:00

closed this issue
added the
pull-request
label

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/ArchiveBox#4367

No description provided.

Rows
Columns