starred/karakeep

Fork 0

mirror of https://github.com/karakeep-app/karakeep.git synced 2026-04-25 07:56:05 +03:00

[GH-ISSUE #1402] Add a bookmark to a list API should be idempotent #889

New issue

Closed

opened 2026-03-02 11:53:32 +03:00 by kerem · 5 comments

kerem commented

2026-03-02 11:53:32 +03:00

Owner

Originally created by @hasansino on GitHub (May 14, 2025).
Original GitHub issue: https://github.com/karakeep-app/karakeep/issues/1402

Originally assigned to: @xuatz on GitHub.

Adding bookmark to the list where it is already present returns 400 error, which prevents (or at least makes it more complicated) for exporters to work.

Originally created by @hasansino on GitHub (May 14, 2025). Original GitHub issue: https://github.com/karakeep-app/karakeep/issues/1402 Originally assigned to: @xuatz on GitHub. Adding bookmark to the list where it is already present returns 400 error, which prevents (or at least makes it more complicated) for exporters to work.

kerem

2026-03-02 11:53:32 +03:00

closed this issue
added the
feature request

status/approved

pri/medium
labels

kerem commented

2026-03-02 11:53:32 +03:00

Author

Owner

@xuatz commented on GitHub (May 17, 2025):

@MohamedBassem It sounds like this can/should be handled together with #864?

I could pick them both up too.

@xuatz commented on GitHub (May 17, 2025): @MohamedBassem It sounds like this can/should be handled together with #864? I could pick them both up too.

kerem commented

2026-03-02 11:53:32 +03:00

Author

Owner

@MohamedBassem commented on GitHub (May 17, 2025):

@xuatz This and #864 are a bit different in my opinion. If you're interesting in working on it, I can assign it to you :)

@MohamedBassem commented on GitHub (May 17, 2025): @xuatz This and #864 are a bit different in my opinion. If you're interesting in working on it, I can assign it to you :)

kerem commented

2026-03-02 11:53:32 +03:00

Author

Owner

@xuatz commented on GitHub (May 17, 2025):

@xuatz This and #864 are a bit different in my opinion. If you're interesting in working on it, I can assign it to you :)

if we have some leeway on timeline then sure 💪

@xuatz commented on GitHub (May 17, 2025): > @xuatz This and #864 are a bit different in my opinion. If you're interesting in working on it, I can assign it to you :) if we have some leeway on timeline then sure 💪

kerem commented

2026-03-02 11:53:32 +03:00

Author

Owner

@xuatz commented on GitHub (May 17, 2025):

@xuatz This and #864 are a bit different in my opinion. If you're interesting in working on it, I can assign it to you :)

@MohamedBassem ohhhh somehow in my mind I thought this issue was referring to
PUT /api/v1/bookmarks/:id 😅

@xuatz commented on GitHub (May 17, 2025): > [@xuatz](https://github.com/xuatz) This and [#864](https://github.com/karakeep-app/karakeep/issues/864) are a bit different in my opinion. If you're interesting in working on it, I can assign it to you :) @MohamedBassem ohhhh somehow in my mind I thought this issue was referring to PUT `/api/v1/bookmarks/:id` 😅

kerem commented

2026-03-02 11:53:33 +03:00

Author

Owner

@MohamedBassem commented on GitHub (May 17, 2025):

@xuatz this is a much simpler one. Just this endpoint (https://docs.karakeep.app/API/add-a-bookmark-to-a-list) needs to not fail when the item is already in the list.

@MohamedBassem commented on GitHub (May 17, 2025): @xuatz this is a much simpler one. Just this endpoint (https://docs.karakeep.app/API/add-a-bookmark-to-a-list) needs to not fail when the item is already in the list.

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/karakeep#889

No description provided.

Rows
Columns