[PR #1211] Fix archive_org icon "exists" #2863

Closed
opened 2026-03-01 18:00:58 +03:00 by kerem · 0 comments
Owner

Original Pull Request: https://github.com/ArchiveBox/ArchiveBox/pull/1211

State: closed
Merged: Yes


Summary

This PR fixes a bug where if you make a snapshot while SAVE_ARCHIVE_DOT_ORG is enabled then the archive_org link will always show as active, despite that being explicitly disabled. This is because of a miscalculation of the "exists" variable which ends up being set to "None" instead of "False". Screenshot included showing this -- it should have class="exists-False" but it's class="exists-None".
20230815_23h24m43s_grim

Related issues

I didn't make an issue for this as when I noticed the problem I just fixed it instead of reporting anything.

Changes these areas

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

Let me know if there's anything I could do to help get this merged.

**Original Pull Request:** https://github.com/ArchiveBox/ArchiveBox/pull/1211 **State:** closed **Merged:** Yes --- <!-- IMPORTANT: Do not submit PRs with only formatting / PEP8 / line length changes. --> # Summary This PR fixes a bug where if you make a snapshot while SAVE_ARCHIVE_DOT_ORG is enabled then the archive_org link will always show as active, despite that being explicitly disabled. This is because of a miscalculation of the "exists" variable which ends up being set to "None" instead of "False". Screenshot included showing this -- it should have class="exists-False" but it's class="exists-None". ![20230815_23h24m43s_grim](https://github.com/ArchiveBox/ArchiveBox/assets/10569817/591b46b5-bb5f-4271-ab6e-dccfbd1bb848) # Related issues I didn't make an issue for this as when I noticed the problem I just fixed it instead of reporting anything. # Changes these areas - [X] Bugfixes - [ ] Feature behavior - [ ] Command line interface - [ ] Configuration options - [ ] Internal architecture - [ ] Snapshot data layout on disk Let me know if there's anything I could do to help get this merged.
kerem 2026-03-01 18:00:58 +03:00
Sign in to join this conversation.
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#2863
No description provided.