Skip to content

Conversation

@skos-ninja
Copy link
Contributor

Upon using the 1Password plugin I've encountered a few problems with error states so I've:

  • Simplified by adding a Result type for return a possible error
  • Changed the fetching function to use the .resolve binding using the secret reference to further reduce the number of api calls
  • Fixed the vault title being missing on newer versions of the 1Password app (it appears they've updated the return value from the app but not the SDK yet)

Copy link
Member

@gschier gschier left a comment

Choose a reason for hiding this comment

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

Nice, thanks for the improvements!

@gschier gschier merged commit 47c5ef1 into mountain-loop:main Jan 9, 2026
2 checks passed
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