Skip to content
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

Plasma 6 port #44

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Plasma 6 port #44

wants to merge 1 commit into from

Conversation

Diyou
Copy link

@Diyou Diyou commented Mar 27, 2024

I made some adjustments to make the widget work with plasma 6.
I had to rework the tableview in the configurations which needs further work but should suffice for the time being if people like to keep using the widget.

Related issue: #43

@hugomelo
Copy link

hugomelo commented Apr 9, 2024

@Diyou can you provide us the plasmoid of this version supporting plasma 6?

@Diyou
Copy link
Author

Diyou commented Apr 10, 2024

You can copy the contents from my fork into ~/.local/share/plasma/plasmoids/com.marcinorlowski.kde.crypto-tracker/
or extract and install:
Crypto-Tracker.zip

Keep in mind when testing, that the CompactRepresentation(on the panel) is no longer supported(maybe this can be fixed in the future) and the generated crypto-data.js is outdated(Some exchanges no longer work incl. the default)

@coffseducation
Copy link

(maybe this can be fixed in the future)

Nice! Is this something that could still happen?
Thanks.

@Diyou
Copy link
Author

Diyou commented Jun 2, 2024

(maybe this can be fixed in the future)

Nice! Is this something that could still happen? Thanks.

Apparently it is possible to add it to the panel. Maybe I had a cache issue back then when making that assumption or something changed with plasma.
Let me know if its working for you.

@coffseducation
Copy link

coffseducation commented Jun 2, 2024

(maybe this can be fixed in the future)

Nice! Is this something that could still happen? Thanks.

Apparently it is possible to add it to the panel.

hmm, it would not add to the panel OR desktop for me. Incompatible.

@hugomelo
Copy link

hugomelo commented Jun 2, 2024

It's working for me using the fork from @Diyou

@coffseducation
Copy link

It installed into the panel with icon but clicking it gave the following:

file:///home/user/.local/share/plasma/plasmoids/com.marcinorlowski.kde.crypto-tracker/contents/ui/main.qml:12:1: module "org.kde.plasma.calendar" is not installed

then after reboot:

`file:///home/anarcho/.local/share/plasma/plasmoids/com.marcinorlowski.kde.crypto-tracker/contents/ui/main.qml:44:22: Type ExchangeContainer unavailable

file:///home/anarcho/.local/share/plasma/plasmoids/com.marcinorlowski.kde.crypto-tracker/contents/ui/ExchangeContainer.qml:14:1: Script file:///home/anarcho/.local/share/plasma/plasmoids/com.marcinorlowski.kde.crypto-tracker/contents/js/crypto.js unavailable

file:///home/anarcho/.local/share/plasma/plasmoids/com.marcinorlowski.kde.crypto-tracker/contents/js/crypto.js:13:1: Script file:///home/anarcho/.local/share/plasma/plasmoids/com.marcinorlowski.kde.crypto-tracker/contents/js/crypto_data.js unavailable

file:///home/anarcho/.local/share/plasma/plasmoids/com.marcinorlowski.kde.crypto-tracker/contents/js/crypto_data.js: No such file or directory`

@Diyou
Copy link
Author

Diyou commented Jun 8, 2024

crypto_data.js needs to be generated by generate_data.py I think... An old one is contained in the zip file posted earlier or you can copy it over from the old plasmoid.

@coffseducation
Copy link

. An old one is contained in the zip file

Thanks. I copied it over, but no change.

@coffseducation
Copy link

SUCCESS! ✅
I am not to sure why.
I had 3 versions in the plasmoids directory so I removed all but one.
After log out/log in I have it working!

Thanks!

@mbol8309
Copy link

when this will be merged?

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.

4 participants