Skip to content

Add Linkding support #895

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

Merged
merged 4 commits into from
Apr 4, 2025
Merged

Conversation

cbos
Copy link
Contributor

@cbos cbos commented Mar 20, 2025

Description

Linkding is a bookmark manager.
With this integration it uses the REST API to fetch some (max 15) bookmarks based on a query.
For each of the returned bookmarks an items is created.
So this is a more dynamic integration.

image

Example use case:
For a project I am working on, I have a number website I often use. I can mark them with a tag in Linkding.
Then query for that tag from Homer, then I have that list at hand.
Once the project is done, I can remove the tag in Linkding and the list is cleaned Homer.
When I use a new website I can easily bookmark that page with the Linkding extension and add the same tag, then it directly appears on the Homer dashboard on new load.

Fixes # (issue)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • I've read & comply with the contributing guidelines
  • I have tested my code for new features & regressions on both mobile & desktop devices, using the latest version of major browsers.
  • I have made corresponding changes to the documentation (README.md).
  • I've checked my modifications for any breaking changes, especially in the config.yml file

Copy link
Owner

@bastienwirtz bastienwirtz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very cool idea @cbos, thanks!
A few minor things and it should be good to go!

@bastienwirtz
Copy link
Owner

Also, could you run a little pnpm lint to make sure everything is formatted ? Thanks!

@cbos
Copy link
Contributor Author

cbos commented Apr 1, 2025

@bastienwirtz
Thanks for reviewing and thanks for the feedback. I handled the comments and resolved the merge conflicts on the the documentation as that was updated in the meantime and causing a merge conflict (which blocked the PR).

Copy link
Owner

@bastienwirtz bastienwirtz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect @cbos , thanks 🙏

@bastienwirtz bastienwirtz merged commit 7cb7293 into bastienwirtz:main Apr 4, 2025
5 checks passed
@cbos
Copy link
Contributor Author

cbos commented Apr 4, 2025

@bastienwirtz
Thanks for merging it!
Do you have an idea when the next release will be created and pushed as container image?

@bastienwirtz
Copy link
Owner

@cbos It's released! thanks again 🙏

@cbos
Copy link
Contributor Author

cbos commented Apr 5, 2025

@bastienwirtz Thanks a lot!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants