mirror of
https://github.com/go-shiori/shiori.git
synced 2026-04-25 06:25:54 +03:00
[GH-ISSUE #18] Unable to put empty string as excerpt #12
Labels
No labels
component:backend
component:builds
component:builds
component:extension
component:frontend
component:readability
database
database:mysql
database:postgres
database:sqlite
feature:ebooks
github_actions
good first issue
hacktoberfest
note:duplicate?
note:fixed?
note:out-of-scope?
os:windows
priority:high
priority:low
pull-request
resolution:as-intended
resolution:cant-reproduce
resolution:duplicate
resolution:fixed
resolution:wontfix
tag:TBD
tag:big-task
tag:help-wanted
tag:huge-data
tag:meta
tag:more-info
tag:next
tag:no-stale
tag:requires-migrations
tag:research
tag:security 🛡️
tag:stale
tag:waiting-for-assignee
type:bug
type:documentation
type:enhancement
type:meta
type:ux
user:cli
user:web
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/shiori#12
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 @sascha-andres on GitHub (Mar 2, 2018).
Original GitHub issue: https://github.com/go-shiori/shiori/issues/18
The excerpt extraction for
https://www.programming-books.io/essential/go/does not work as expected.It displays
↑ ↓ to navigate ↵ to select Esc to closeand even when I edit the excerpt to have it empty, it fills in the excerpt again.My expected behaviour would have been to be able to set the excerpt explicitly to an empty string
@peteretelej commented on GitHub (Mar 2, 2018):
Empty excerpts seem to be overwritten during update
github.com/RadhiFadlillah/shiori@3826d4b7ab/cmd/update.go (L149-L151)whitespace characters are also trimmed during input, hence will be overwritten.
github.com/RadhiFadlillah/shiori@3826d4b7ab/view/index.html (L74)also applies for the title. I think deleting the excerpt at least would be nice, maybe an additional table column for deletedexcerpt, or using a character like '-' for deleted excerpts and replacing when displaying/reading
@RadhiFadlillah commented on GitHub (Mar 6, 2018):
Fixed in 9d3c8ee.
It's quite a dirty hack, however it's simple and works, so I think it's good enough for now.