| dcaliste | Hello chriadam, how are you ? | 07:02 |
|---|---|---|
| chriadam | hi dcaliste, I'm ok thanks. How are you? | 07:02 |
| dcaliste | I'm fine, thanks. | 07:04 |
| chriadam | great | 07:04 |
| dcaliste | I'm looking at the comments you added to the PRs. | 07:05 |
| chriadam | thanks for all of your efforts recently! | 07:05 |
| chriadam | no comments of substance, yet | 07:05 |
| chriadam | I merged the mkcal PR and tagged that one | 07:05 |
| chriadam | I saw the caldav unit tests PR, but haven't yet had time to digest the changes properly. | 07:07 |
| chriadam | also caldav PR#7 with the minor error code change, that one looks good to me, wondering if there is a way to easily test that "item failure" case in reality? | 07:08 |
| dcaliste | I added a comment. The easiest way is to import an orphan exception via ICS data. | 07:09 |
| chriadam | thanks | 07:09 |
| dcaliste | It will showcase of of the different case : a failing upload because of error in the database. | 07:10 |
| chriadam | yep, I guess it cannot generate the series for upsync | 07:10 |
| dcaliste | For network related issues, it's more tricky without playing with the DB. | 07:10 |
| chriadam | yeah. | 07:11 |
| dcaliste | Indeed, it fails because the ICS data cannot be generated for upload, missing parent. | 07:11 |
| chriadam | aside from those two new caldav PRs, were there any other new PRs which need review? | 07:11 |
| chriadam | I guess the two n-q-p-c PRs are almost ready for merge, just would like pvuorela to take a quick look at those also | 07:11 |
| dcaliste | That's all for the new PRs. | 07:12 |
| chriadam | great. thanks very much, again. | 07:13 |
| chriadam | regarding the week-view one, I saw that pvuorela has been discussing that one with you | 07:13 |
| dcaliste | You have the jolla-calendar #309 that goes with https://github.com/sailfishos/nemo-qml-plugin-calendar/pull/7 | 07:14 |
| chriadam | hopefully that is all going smoothly | 07:14 |
| chriadam | oh! thanks, somehow I missed that | 07:15 |
| dcaliste | Because the nemo-qml-plugin is changing the behaviour to overwrite instead of crashing. So we need to warn the user of the action. | 07:15 |
| chriadam | did I create a bug report for this case already? | 07:16 |
| dcaliste | Well, not crashing sorry, but ffailing to import. | 07:16 |
| dcaliste | There is a forum entry. Let me find it. | 07:16 |
| dcaliste | https://forum.sailfishos.org/t/bug-4-2-0-21-cannot-import-calendar-entries-via-ics-file/7988/13 | 07:17 |
| chriadam | is this like "Importing multiple instances of same calendar event (with different REVISION) fails" ? | 07:17 |
| chriadam | yes, this one | 07:17 |
| chriadam | ok there is JB#55786 | 07:17 |
| dcaliste | For different revisions, I'm not sure. I can check it. And add the bug id. | 07:18 |
| chriadam | thank you | 07:19 |
| chriadam | (either way, that bug number will do, as it's about import failures, and links to that forum link in the bug, so it's definitely related) | 07:19 |
| dcaliste | I will also look at pvuorela PR in mKCal this week. | 07:20 |
| chriadam | thank you. it's basically moving toward a "schema versioning / upgrade" system | 07:21 |
| chriadam | similar to what is used in qtcontacts-sqlite, I guess. | 07:21 |
| dcaliste | Indeed, I gave a quick overview last week, but I wanted to spend a bit more time on it. | 07:22 |
| chriadam | thanks very much | 07:22 |
| chriadam | is there anything else to discuss this week? @flypig did you have anything? | 07:23 |
| dcaliste | There is also jolla-calendar #310. Pvuorela already gave a deep look. So it's on going work. | 07:23 |
| chriadam | yes, I think he needs to discuss the license question with management, I guess | 07:23 |
| chriadam | the feature itself is looking great already | 07:24 |
| dcaliste | Yeh. That would be great if it could be BSD (or other) in the main repo. | 07:24 |
| chriadam | yep, let's see | 07:24 |
| dcaliste | There still the possibility to create component package with a dedicated license. | 07:24 |
| dcaliste | But Sailfish.Calendar is already used... | 07:24 |
| dcaliste | And it would just have the week view, hich seems a bit overkill. | 07:25 |
| chriadam | hopefully it's not required | 07:25 |
| chriadam | anyway, let's see :-) | 07:25 |
| dcaliste | Ok, indeed let see and discuss when there are news in that direction. | 07:26 |
| dcaliste | There is maybe one last think : https://github.com/sailfishos/contactsd/pull/1 mostly targetting at flypig. | 07:27 |
| chriadam | thanks let me check | 07:27 |
| dcaliste | I cannot reproduce what some users are reporting on the forum which makes the PR a bit fragile. | 07:27 |
| chriadam | yeah, having no consistent repro is unfortunate | 07:28 |
| chriadam | but a speculative fix is fine so long as we're confident that it won't cause a regression, of course | 07:29 |
| chriadam | I pinged flypig about that, I guess he hasn't had a chance to review that one yet, unfortunately. | 07:29 |
| flypig | Apologies dcaliste, I didn't get a chance to look at that yet. | 07:29 |
| dcaliste | Yeh, that's the main problem here, regression could be not very far. | 07:29 |
| dcaliste | flypig, ok, no problem. | 07:29 |
| dcaliste | So we can discuss this one at a later meeting when you will have time to look at it. No problem of course. | 07:32 |
| chriadam | thanks, let's do that | 07:32 |
| chriadam | anything else to discuss this week? | 07:32 |
| chriadam | if not - thanks very much for your efforts as usual, dcaliste. I will poke pvuorela again about those two nqpc PRs hopefully get those in this week. | 07:33 |
| dcaliste | Indeed, that's all. Thank you also for the support you provide, all of you, pvuorela, flypig, yourself... | 07:34 |
| chriadam | have a great week! | 07:34 |
| * chriadam -> away | 07:34 | |
| dcaliste | thank you, for you also. | 07:34 |
| poetaster | anyone an idea how I debug /org/nemomobile/lipstick/screenshot rebooting volla? | 12:46 |
| poetaster | nothing in logcat dmesg. | 12:46 |
| Thaodan | poetaster: journalctl? | 16:48 |
| Thaodan | you can check pstore if there is something | 16:49 |
| hnj | does anyone here use piperka client? does it currently work for you? I’m getting an error "SSL Handshake failed" that I’m trying to fix. Any advice? | 19:25 |
| hnj | It has started around the time LetsEncrypt’s root certificate expired but I don’t know whether that’s the cause. Any advice how to find out? | 19:26 |
| ender | can you open the site with the Browser? | 19:51 |
| hnj | ender: yes | 19:53 |
| ender | hm | 20:00 |
| ender | I know I saw root_cert_expired (or something very similar) in the browser once, but it seems to have fixed itself on it's own | 20:01 |
| ender | (that is, just after the Let's Encrypt intermediate expired recently) | 20:01 |
| hnj | would anyone be willing to test the piperka client from the jolla store on their device and let me know if it works? | 20:22 |
| poetaster | thaodan: journalctl froze, as did logcat. nothing. | 20:23 |
| ender | hnj: seems to work for me | 20:33 |
| hnj | ender: sailfish 4? | 20:50 |
| ender | 4.2.0.21, Xperia 10 | 20:56 |
| hnj | ender: ok, thanks | 20:59 |
| Thaodan | poetaster: you can boot to recovery after that happend, ount pstore and check /sys/fs/pstore | 21:31 |
Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!