Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Creating or editing a bookmark doesn’t create new tags #27

Closed
marmata opened this issue Apr 27, 2023 · 7 comments · Fixed by #28
Closed

Creating or editing a bookmark doesn’t create new tags #27

marmata opened this issue Apr 27, 2023 · 7 comments · Fixed by #28
Assignees
Labels
bug Something isn't working
Milestone

Comments

@marmata
Copy link

marmata commented Apr 27, 2023

Hi! Just begin using link ding and this beautiful app, fantastic companion! After adding some bookmarks just realized that new tags are not created, is this intentional as in not implemented yet?

@amoscardino amoscardino self-assigned this Apr 27, 2023
@amoscardino amoscardino added the bug Something isn't working label Apr 27, 2023
@amoscardino
Copy link
Owner

Tags, new and existing, are sent to linkding when creating or updating a bookmark. This should result in the app's list of tags being refreshed, but I have noticed that this does not always happen. I will investigate more.

@marmata
Copy link
Author

marmata commented Apr 27, 2023 via email

@amoscardino
Copy link
Owner

Strange. Can you send an example tag value that does this? It may be an issue with the linkding API or how the tags are formatted before they are sent.

@marmata
Copy link
Author

marmata commented May 2, 2023 via email

@amoscardino
Copy link
Owner

I think I have a fix for this. It appears that the tags don't persist back to the bookmark object to be saved until you move focus from the input. So having the input focused and then tapping the save button doesn't have a chance to read those changes before sending the bookmark off to the API. I'm testing this fix now but I imagine I'll have it to app review before the end of the week.

@marmata
Copy link
Author

marmata commented May 10, 2023 via email

@amoscardino amoscardino added this to the v1.2 milestone May 11, 2023
@amoscardino amoscardino mentioned this issue May 12, 2023
@marmata
Copy link
Author

marmata commented May 12, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants