Add feature to locally view ICS/VCAL files about gcalcli HOT 8 CLOSED

I think this is a good idea. Just having the "-v" option dump out the event details before doing any google calendar magic will be a nice change.

from gcalcli.

Oh yeah - that's the best solution. I w as actually thinking about adding a different command. But this idea is much better. The import -v option doesn't need to do any Google stuff unless the user actually chooses [i]mport.

from gcalcli.

I've had a go at patching this myself. I couldn't get it to work using just the -v option, because there could be mulitple events, and after printing the details of each event, there is the option to import - so it's not that simple.

So instead I added an import -d option to dump the ICS details and exit. I couldn't use ImportICS / CreateEventFromVOBJ as it was because they require the gcalcli class (so the google authentication must be done), so I moved the verbose output parts into a common function just to print the event details. It's not ideal, since this duplicates the logic in CreateEventFromVOBJ, but couldn't think of a better way to do.

I'm sure you guys can do a better job, but I can send a pull request if you want?

from gcalcli.

Send a pull request and we can review. I've been making a number of changes
to that code anyway to support The icalendar library and other formats.

On Wednesday, February 27, 2013, Michael Maddern wrote:

I've had a go at patching this myself. I couldn't get it to work using
just the -v option, because there could be mulitple events, and after
printing the details of each event, there is the option to import - so it's
not that simple.

So instead I added an import -d option to dump the ICS details and exit. I
couldn't use ImportICS / CreateEventFromVOBJ as it was because they require
the gcalcli class (so the google authentication must be done), so I moved
the verbose output parts into a common function just to print the event
details. It's not ideal, since this duplicates the logic in
CreateEventFromVOBJ, but couldn't think of a better way to do.

I'm sure you guys can do a better job, but I can send a pull request if
you want?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/63#issuecomment-14175035
.

from gcalcli.

Ok I've sent pull request.

I've been having problems importing some ICS files (not related to changes I've made). I was going to raise a defect for it, but I can leave it for now if you're making changes to how this works, and retest it again after.

from gcalcli.

There are known issues with certain ICS files that come from vobject. For example a comma in a description actually has special meaning that vobject holds strictly to. I am working on a branch that uses icalendar, but that has it's own set of special issues, just seems to be a little more actively developed.

from gcalcli.

Does the new -d option implemented in #67 resolve this for you?

nm. #67 is your pull request so I'm going to go with yes ;-)

from gcalcli.

Sorry for replying to this so late, but I only just pulled these changes. The -d option is working great. Thanks!

from gcalcli.