mirror of
https://github.com/ArchiveBox/ArchiveBox.git
synced 2026-04-25 17:16:00 +03:00
[GH-ISSUE #857] Bug: 500 error when attempting to sort by size #529
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#529
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?
Originally created by @WesleyAC on GitHub (Sep 26, 2021).
Original GitHub issue: https://github.com/ArchiveBox/ArchiveBox/issues/857
Steps to reproduce
Screenshots or log output
ArchiveBox version
@pirate commented on GitHub (Sep 27, 2021):
Yeah I haven't implemented sorting by size, so I'm not surprised it's failing. I might just remove the sort ability for now, as it's not an easy problem (size is not a DB column, it's computed lazily and cached).
@WesleyAC commented on GitHub (Sep 28, 2021):
Makes sense. Sorting by size would be nice, but it's not hard to
du -s archive/* | sort -g, so I don't really mind not having it much — it'd be better for it to be disabled than 500 :)@pirate commented on GitHub (Jan 19, 2024):
Fixed in
861d44da98. Will be released in v0.7.3The new behavior is to just sort by number of archive results, since size is still not a db column at the moment.