search

forslund / gcalendar_skill

Mycroft skill for google calendar
GNU General Public License v3.0
21 stars 12 forks source link

Nothing annotated on my skills page #3

Open bgarr33434 opened 6 years ago

bgarr33434 commented 6 years ago

I don't have the option to click on "connect" after successfully installing gcalendar_skill pi@picroft:/opt/mycroft/skills $ sudo msm install https://github.com/forslund/gcalendar_skill.git INFO - Downloading skill: https://github.com/forslund/gcalendar_skill.git INFO - Installing requirements.txt for gcalendar_skill INFO - Successfully installed gcalendar_skill

forslund commented 6 years ago

There was some python 2 only code that hadn't been updated which hindered the skill from starting. I've updated the code to fix this. If you do an upgrade it should work.

bgarr33434 commented 6 years ago

just an fyi. I did a fresh install of picroft today, and I still lost credentials.

forslund commented 6 years ago

Hi, the skill hasn't been updated for the latest changes in mycroft-core so it's currently not in a functioning state. I'll be pushing an update with fixes for you in a day or two.

webimpression commented 4 years ago

Hey Forslund! First of all, thanks for the cool skills. You are doing an amazing job.

I also can't find the credentials on the skills page.

forslund commented 4 years ago

Hi,

does the skills show up at all on the skills page?

webimpression commented 4 years ago

Ok, got it to work. Thanks for wanting to help out but this was a weird local thing. Maybe it's something that can be fixed though. -> Had a look at requirements.txt -> Looked in the log Problem -> Saw some error regarding tzlocal "ModuleNotFoundError: No module named 'tzlocal'" and this message "Failed to launch package manager: Sudo required (and script not launched interactively)" Solution -> home/.../mycroft-core/.venv/bin/python -m pip install tzlocal

(Just writing you this in case it might help. It sure did for me.)

forslund commented 4 years ago

Ah right thanks. will look at that dependency. Should either be added to requirements.txt or removed from the skill

forslund commented 4 years ago

Ok looking at the WIP branch for caldav I've removed the use of that dependency so I'll pull that into master and fix that part (after verifying that all looks ok still)