mirror of
https://github.com/go-shiori/shiori.git
synced 2026-04-25 14:35:52 +03:00
[GH-ISSUE #148] ability to edit the old url's? #111
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#111
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 @thepenguinthatwants on GitHub (Sep 2, 2019).
Original GitHub issue: https://github.com/go-shiori/shiori/issues/148
I sometimes update certain url's but seems like Shiori is unable to edit a URL once entered.
@RadhiFadlillah commented on GitHub (Sep 5, 2019):
Hi @thepenguinthatwants, edit URL is supported in latest version of Shiori.
From cli you can use
--urlflag inupdatecommand like this :In web interface you can modify the URL in edit dialog :
@thepenguinthatwants commented on GitHub (Sep 5, 2019):
Hi!
Oh cool.
I am using the docker image and it hadn't have this feature yet.
@RadhiFadlillah commented on GitHub (Sep 5, 2019):
@thepenguinthatwants yeah, I haven't updated the docker image in a while.
I will try to update docker image ASAP.
@thepenguinthatwants commented on GitHub (Sep 5, 2019):
Thanks!
There are quite many that uses the docker image. Really thankful that about this application.
@RadhiFadlillah commented on GitHub (Sep 21, 2019):
Hi @thepenguinthatwants, the Docker image has been updated and can be pulled by running :
@thepenguinthatwants commented on GitHub (Sep 22, 2019):
Hi!
Thanks it looks good. But how to edit entries? I cant see any options or clickable things for editing the information? Feels like all I can do is about opening the url.
@thepenguinthatwants commented on GitHub (Sep 22, 2019):
Actually after the update it seems like my user has lost all the rights?
@hoijui commented on GitHub (Sep 22, 2019):
same here.
(I am using latest git version, can easily test new stuff if you have something to test)
@RadhiFadlillah commented on GitHub (Sep 22, 2019):
@thepenguinthatwants @hoijui I'm so sorry for the trouble 😞
This issue happened because in Shiori v1.0.0 there are no account level, which means everyone is treated as owner. However, Shiori v1.5.0 has two account level i.e. owner and visitor. The level difference is stored in database as boolean value in column
ownerwith default valuefalse(which means by default all account is visitor, unless specified otherwise)github.com/go-shiori/shiori@89aad30a17/internal/database/sqlite.go (L41-L48)Because in v1.5.0 by default all account is visitor, when updating from v1.0 to v1.5 all of the old accounts by default will be marked as visitor. Fortunately, when there are no
ownerregistered in database, we can login as owner using default account :So, as workaround for this issue, you should :
@thepenguinthatwants commented on GitHub (Sep 22, 2019):
So existing users cant be changed into admins?
Also is it possible to delete the shiori account?
@RadhiFadlillah commented on GitHub (Sep 22, 2019):
Yeah, for now I haven't add the menu to change account level, so the workaround is delete it then create a new one with same name.
The
shioriaccount will be disabled once there is at least one owner account registered in database, as can be seen in these lines.@RadhiFadlillah commented on GitHub (Oct 4, 2019):
Since it seems this issue has been solved, I will close it. Feel free to reopen it if needed.