-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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 keepassxc extension #17758
base: main
Are you sure you want to change the base?
Update keepassxc extension #17758
Conversation
- chore(CHANGELOG): add logs for 1.5.5. - fix: handle the invalid credentials error message. - chore(npm): update packages. - chore: remove test guide. - docs(assets): update screenshot for unlocking database. - feat(unlock): improve UI for info. - chore: add .vscode to .gitignore. - docs(README): fix link for image. - chore: delete \'.vscode\'. - Initial commit
Thank you for your contribution! 🎉 🔔 @zkytech 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. Due to our current reduced availability, the initial review may take up to 10-15 business days |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
This PR updates the KeePassXC extension with improved error messaging, documentation changes, and dependency updates.
- There's a spelling error in CHANGELOG.md: "explicite" should be "explicit" in the error message description
- The error message in
keepass-loader.ts
removes specific error codes which could make debugging harder, consider keeping detailed errors in logs - The security message in
unlock-database.tsx
is now consolidated but could be more prominent given its importance - Dependencies
@raycast/api
and@raycast/utils
have been updated to newer minor versions - The README image source now correctly uses repository-relative paths instead of external URLs
💡 (1/5) You can manually trigger the bot by mentioning @greptileai in a comment!
6 file(s) reviewed, 3 comment(s)
Edit PR Review Bot Settings | Greptile
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
It's because the mapping data in the extensions/.github/raycast2github.json Line 1830 in 201037d
|
I've done a commit to change the mapping 4b6df1b. Edit: 381a6db to merge conflicts. |
Description
Hello 👋🏻,
Here is what I've done:
Changed
keepassxc-cli
command.Fixed
Removed
Checklist
npm run build
and tested this distribution build in Raycastassets
folder are used by the extension itselfREADME
are placed outside of themetadata
folder