Skip to content
This repository has been archived by the owner on Jan 26, 2023. It is now read-only.

Need updation of the plugin as per new albert version 0.18 #1

Closed
prashantjain25 opened this issue Jan 9, 2023 · 9 comments · Fixed by #2
Closed

Need updation of the plugin as per new albert version 0.18 #1

prashantjain25 opened this issue Jan 9, 2023 · 9 comments · Fixed by #2

Comments

@prashantjain25
Copy link

prashantjain25 commented Jan 9, 2023

With new updates to version 0.18.10 the unit-converter is not loading in plugins section, therefore it might require update.

@DenverCoder1
Copy link
Owner

Let me know if it's working for you now.

Check the new instructions added to the readme.

@DenverCoder1
Copy link
Owner

It seems that Albert 0.18 isn't really liking plugins with no prefix trigger so much. While it is working as before currently, it's possible the synopsis may change in a future version.

@prashantjain25
Copy link
Author

prashantjain25 commented Jan 10, 2023

converter seems working now only when calculator is not working as from below extensions:-
https://github.com/albertlauncher/python

Is there currency conversion also in plan? units could be better for integration.

@DenverCoder1
Copy link
Owner

There was a currency converter already in albert (by Manuel) but it seems it's not up to date with 0.18.

https://github.com/albertlauncher/python/blob/master/.archive/currency_converter/__init__.py

I suppose I could merge the two plugins into one.

@prashantjain25
Copy link
Author

prashantjain25 commented Jan 10, 2023

Also seems calculator and converter are clashing, calculator stops working when converter is enabled.

@DenverCoder1
Copy link
Owner

Yeah, I think I'll need to give a default prefix, for example "convert ..."

@DenverCoder1
Copy link
Owner

Do you think it would be better to default to "uc " or "convert "

"convert" is more natural, but "uc" is a bit more convenient since it's short

@prashantjain25
Copy link
Author

yes shorthand notation would be appealing to everyone like "cnv"

@DenverCoder1
Copy link
Owner

I added currency support. You can configure the trigger in the settings to be cnv or whatever you like.

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

Successfully merging a pull request may close this issue.

2 participants