-
Notifications
You must be signed in to change notification settings - Fork 17
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
error in searching keepassxc database #26
Comments
I made a pr fixing that. Also, is your search slow? it takes like 3 seconds after typing to show results. |
A little slow, but i have a huge database with about 800 items. A search takes 1-3 seconds on my laptop. Glad you are on top of this, as i use keepass for all my accounts and the extension makes it much more efficient
Benji
…----------------------------------------
Apr. 16, 2022 03:19:33 default-303 ***@***.***>:
I made a pr fixing that.
Also, is your search slow? it takes like 3 seconds after typing to show results.
—
Reply to this email directly, view it on GitHub[#26 (comment)], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AHXMRESQUFGRKU3X2VIDZB3VFJSYFANCNFSM5SMQKD4A].
You are receiving this because you authored the thread.[Tracking image][https://github.com/notifications/beacon/AHXMRETK6GLO3JHZPBBEQE3VFJSYFA5CNFSM5SMQKD4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOIGM3RSA.gif]
|
For those interested, it's #27 |
@pbkhrv when is it planned that this PR will be merged? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
ulauncher-keepassxc/keepassxc/keepassxc_db.py
Line 127 in 11309e2
I attached a screenshot of the error I am getting in the last few days when trying to search my keepass database. The error is resolved by changing the word "locate" to "search" in the referenced line
.
The text was updated successfully, but these errors were encountered: