-
Notifications
You must be signed in to change notification settings - Fork 210
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
[WIP] Add preliminary support for VTODOs #1118
base: master
Are you sure you want to change the base?
Conversation
Quick PoC that converts VTODOs to VEVENTs before adding them to the backend, this enables us to not treat tasks as something very special. If this approach is interesting enough, it should be forbidden that khal edits tasks as it is out of its scope. This would fix pimutils#448
for more information, see https://pre-commit.ci
Forgot to make it explicit that: "convert VTODOs into VEVENTs" only applies to the in-memory representation while importing the data, since what is saved into sqlite3 is the raw text and what is used for representation comes from there as well, this is fully compatible with the represented object being a VTODO in the underlying vdir. Since we would forbid |
@evilham sounds interesting. Not sure why but I am unable to get this working. I created a test task using todoman:
But Khal does not show it:
This is the VTODO entry in my calendars folder:
Potential points of problem:
Also I verified I am using the correct branch:
|
Thanks for the testing! I don't think Japanese has anything to do with it :-D most likely I missed a code path that was breaking; could you try again? If that doesn't work, could you check if there is a higher verbosity setting for khal that might give us more info? |
@evilham working nice:
Also in interactive view:
|
That's great! please report any oddities and do not try to edit tasks with |
Hey, I'm evaluating
khal
and as others have mentioned, VTODOs should really be listed.Since this is a must for me, I created a
quick PoC that converts VTODOs to VEVENTs before adding them to the
backend, this enables us to not treat tasks as something very special.
If this approach is interesting enough, it should be forbidden that
khal edits tasks as it is out of its scope.
This would fix #448
I noticed that the sqlite3 database should be deleted in order to force
khal
to process any pre-existing VTODOs (probably just a matter of the vdir being 'up-to-date'). Don't think that's an issue.This is the output with a recurrent meeting and a task:
And when the task has been marked as done: