-
Notifications
You must be signed in to change notification settings - Fork 314
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
Import fails, seemingly because of no end time #486
Comments
Thanks for the report and the suggested fixes. I haven't used the ICS import stuff very much myself. So I don't have a strong opinion about how to proceed. We already do have a lot of prompts in our UX, so adding the prompt for user input fits with what we're already doing. I don't want to pop up "windows", however, since part of the point of gcalcli is that you can use it in a non-graphical environment. Basically, if someone who uses the ICS stuff brings forward an implementation of (2), (3) or (4), I'd be looking to merge it. |
Does my #282 "CreateEventFromVOBJ: if there is no dtend, use the duration if available" help? |
I'm quite behind on reviewing that. Sorry! @jordanmendler, have you tried that branch to see if it's an improvement over master? |
4 years later ...
I had the same problem. And your PR solved it. |
I have an ICS file that was given by my healthcare provider (UCLA). This seems to have been generated through Epic, which is one of the main ERP systems for healthcare, as almost every hospital or clinic I have been to uses Epic regardless of which state. When I try to import it I get the error "Error: event does not have a dtstart and dtend!" Doing some debugging, DTSTART it indeed present in the ICS, but DTEND is not.
I can, however, successfully import it through Google's import page, and when they add the event to my calendar they just set the duration to 1 hour.
As such, rather than failing with an ambiguous message, I'd propose one of the following, in order of what I see as best to worst-case solution in terms of UX:
Here is the ICS with some information redacted for privacy:
And here is the execution with some info redacted for privacy:
The text was updated successfully, but these errors were encountered: