[GH-ISSUE #675] Save hasEbook, hasArchive, etc... in database #339

Open
opened 2026-02-25 23:33:59 +03:00 by kerem · 2 comments
Owner

Originally created by @Monirzadeh on GitHub (Jul 23, 2023).
Original GitHub issue: https://github.com/go-shiori/shiori/issues/675

In this specific situation, we call directly bookmark from database. So we don't get any specific parameter from request or current run (like has. Archive or hasEbook) and we don't save that parameter on database (legacy code). This parameter defines each time you open homepage of shiori
If browser, just open /bookmark/:id/ebook file should be present to the browser regardless of request, as it should be.(just check user access).
If my explain is a little confusing, please look at this part
github.com/go-shiori/shiori@e839b69875/internal/webserver/handler-api.go (L201C1-L214C3)
If it is not a concern right now, we can set resolve and decide about that in #497
if not, do you have any idea as better handle that?
Same thing happen in archive and content but it on #497 and keep commit related to feature.

Originally posted by @Monirzadeh in https://github.com/go-shiori/shiori/pull/623#discussion_r1250691272

If users remove the file from disk manually, should we update the value in the database automatically, or it should be something just in dev mode?

Originally created by @Monirzadeh on GitHub (Jul 23, 2023). Original GitHub issue: https://github.com/go-shiori/shiori/issues/675 >In this specific situation, we call directly bookmark from database. So we don't get any specific parameter from request or current run (like `has. Archive` or `hasEbook`) and we don't save that parameter on database (legacy code). This parameter defines each time you open homepage of shiori If browser, just open /bookmark/:id/ebook file should be present to the browser regardless of request, as it should be.(just check user access). If my explain is a little confusing, please look at this part https://github.com/go-shiori/shiori/blob/e839b6987580b750103304258ac8385f1163de08/internal/webserver/handler-api.go#L201C1-L214C3 If it is not a concern right now, we can set resolve and decide about that in #497 if not, do you have any idea as better handle that? Same thing happen in archive and content but it on #497 and keep commit related to feature. _Originally posted by @Monirzadeh in https://github.com/go-shiori/shiori/pull/623#discussion_r1250691272_ If users remove the file from disk manually, should we update the value in the database automatically, or it should be something just in dev mode?
Author
Owner

@stale[bot] commented on GitHub (Aug 29, 2023):

This issue has been automatically marked as stale because it has not had any activity for quite some time.
It will be closed if no further activity occurs.
Thank you for your contributions.

<!-- gh-comment-id:1698024133 --> @stale[bot] commented on GitHub (Aug 29, 2023): This issue has been automatically marked as stale because it has not had any activity for quite some time. It will be closed if no further activity occurs. Thank you for your contributions.
Author
Owner

@Monirzadeh commented on GitHub (Aug 30, 2023):

we need to keep it open

<!-- gh-comment-id:1698580020 --> @Monirzadeh commented on GitHub (Aug 30, 2023): we need to keep it open
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/shiori#339
No description provided.