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

Update linkding extension - close issue + loads of minor enhancements #17043

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

xmok
Copy link
Contributor

@xmok xmok commented Feb 13, 2025

EDIT: for testing, reviewers can use the demo version @ https://demo.linkding.link/login/

Description

Fixes & Enhancements

General

Create Bookmark

  • useForm hook for control
  • Add placeholders and info where relevant

Search Bookmarks

  • make "Delete" ActionStyle Destructive
  • fix: after delete, Success Toast was always shown even on fail
  • Show Toasts throughout "Delete" in search-bookmarks
  • Show proper "isLoading" in search-bookmarks (usePromise hook)
  • Add Icon to search-bookmarks items

Manage Account

  • Action to open manage-account when no accounts added
  • useForm hook in manage-account for better form control
  • Add Icons in manage-account

Screencast

Since a lot of changes are QoL, it is difficult to compare them all so the screencast 1st open Store version then goes through the updated one.

linkding.mp4

Checklist

@raycastbot raycastbot added extension fix / improvement Label for PRs with extension's fix improvements extension: linkding Issues related to the linkding extension labels Feb 13, 2025
@raycastbot
Copy link
Collaborator

raycastbot commented Feb 13, 2025

Thank you for your contribution! 🎉

🔔 @InteractiveNinja you might want to have a look.

You can use this guide to learn how to check out the Pull Request locally in order to test it.

You can expect an initial review within five business days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
extension fix / improvement Label for PRs with extension's fix improvements extension: linkding Issues related to the linkding extension
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Linkding] The setup should tolerate trailing slashes on the URL
2 participants