mirror of
https://github.com/ArchiveBox/ArchiveBox.git
synced 2026-04-25 17:16:00 +03:00
[PR #1314] Show the upgrade notification only in specific views #2886
Labels
No labels
expected: maybe someday
expected: next release
expected: release after next
expected: unlikely unless contributed
good first ticket
help wanted
pull-request
scope: all users
scope: windows users
size: easy
size: hard
size: medium
size: medium
status: backlog
status: blocked
status: done
status: idea-phase
status: needs followup
status: wip
status: wontfix
touches: API/CLI/Spec
touches: configuration
touches: data/schema/architecture
touches: dependencies/packaging
touches: docs
touches: js
touches: views/replayers/html/css
why: correctness
why: functionality
why: performance
why: security
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/ArchiveBox#2886
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Original Pull Request: https://github.com/ArchiveBox/ArchiveBox/pull/1314
State: closed
Merged: Yes
Summary
This PR adds back the upgrade notification banner that was originally added in #1274 and removed in
4af9bee, but only to specific views instead of the entire admin site, as discussed in #1306.I wasn't sure for which views we should show the banner, so for now it's only showing on the snapshot view (the page at
/admin/core/snapshot/.The necessary context is provided by overriding the
changelist_viewmethod of theSnapshotAdminmodel. I think this could work for all views except for the main admin page (/admin/). If we want to pass context to the view for that page, I think we can override theeach_contextmethod ofArchiveBoxAdmininstead.Related issues
#696
Changes these areas