-
-
Notifications
You must be signed in to change notification settings - Fork 27
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
BUG: Shared event (where I am a guest) is showing up a day early in my daily notes #98
Comments
Thanks for the bug report @dlhawe . Is there anything else you can add to help debug this? Maybe the ICS VEVENT entry, anonymized? To do that, download the ICS file to an editor, and find the VEVENT entry, and copy/paste it in? The parsing is entirely done through |
Thanks for your speedy reply. Im not a frequent github user, would you prefer it pasted here or attached (if that is even possible)? Also, I will post two vevents as I am not sure which is the one from the creator and which is mine if that is ok. (EDIT: You said copy paste, sorry missed that)
|
Thank you so much, I'll take a look and see if I have any ideas! Probably over the weekend. Diagnosing this stuff is very hard when I don't have a VEVENT to reproduce the issue with. |
Can you add @dlhawe , which events are coming through on the wrong day? i.e. which day did it show up in, when did you expect it? That'll help me when I try to reproduce the issue. |
So those two VEVENT items are two views of the same calendar event as my wife has shared her calendar with me. One of the events is my wife's event (creator) on her calendar, the other is the same event on my calendar (guest). Not sure which event is which above. The event is scheduled for 17:00 today (2024-01-04) at 17:00 and shows up properly on my phone and calendar web page. This event showed up on yesterdays (2024-01-03) daily note and did not show on today's daily note. Please let me know if showing you the code I am using to pull the event on my daily calendar would be of use, I can post that as well. Thanks for having a look, and no rush. It's a mild annoyance and certainly not a deal breaker for a fantasic plugin. |
@dlhawe I am confused but may see the issue
Because of #19 , this could explain the apparently erroneous entry yesterday. That doesn't explain why it didn't show up today though. |
So I am a bit confused (and embarrassed). Neither of these events are the event from today. I just posted the last two entries from the reoccuring event. I apologize for that. Given that I wasnt importing calendar events on 20231019 (first VEVENT), I can't comment on that. TBH, I didn;'t notice if the 20231228 event posted early, but I can confirm I did decline it. The confusing part is that I don't have an entry in my ICAL dump for the 20240104 event that posted to yesterdays daily note, nor for any of the future reoccuring events in that series as far as I can tell which makes diagnosing this issue problematic. I am also noticing that reoccuring future events I have personally posted dont show on the ICAL dump either. The more I post about this, the dumber I sound. Sorry. |
Don't be embarrassed! Recurring rules are strange beasts and supremely counterintuitive. You might be able to find the event by searching for the Google meet link. There should then be an rrule section below it, that's where the instances are declared and the client has to compute them. |
So I was able to get the desired event to export by switching my attendence from yes to maybe, then back to yes. That seemed to update the event and allow it to export. Reminder: This event was today (2024-01-04), and it a reoccuring event every Thurs at 5pm PST. I am a guest on this reoccuring event. This specific event imported a day early (2024-01-03) onto my daily note.
|
Forgot to update this...
Surprisingly, it shows up twice. |
With that, I am at a loss as to how to debug this further. I'll see if I can figure out why it's getting listed twice. |
This was a separate issue, fixed in #100. |
Thank you for testing, I sincerely appreciate it. It seems to me that it is either an issue with me being a guest on my wife's event and/or the fact it is is reoccuring event. I have a reoccuring event that starts on Monday that I created, and I'll provide feedback if desired as to if that shows up properly. |
I suspect the recurring event aspect because I can't tell you how much trouble recurring events (and timezones!) have given me. ;) If it is related to the guest thing, maybe #101 (once merged) will give us a hint as to what's going on as we'll soon be able to see guests on events.
Thanks so much! I want to get this bug quashed. |
FYI - I am seeing a similar issue on my google calendar. Only on recurring events - both on shared calendar as well as a personal calendar. A latest example is I have an event tomorrow evening, but it is showing up on my obsidian today when using this plugin |
Thanks for the update. Can you share more details? I haven't been able to replicate this and since I leverage |
I see this message in my obsidian developer console
RRULE origOptions: {
"tzid": "America/Los_Angeles",
"dtstart": "2024-01-09T04:00:00.000Z",
"freq": 2,
"wkst": {
"weekday": 6
},
"until": "2024-03-06T07:59:59.000Z",
"byweekday": [
{
"weekday": 0
}
]
}
{
"type": "VEVENT",
"params": [],
"start": "Sun Jan 14 2024 20:00:00 GMT-0800 (Pacific Standard Time)",
"datetype": "date-time",
"end": "Sun Jan 14 2024 21:00:00 GMT-0800 (Pacific Standard Time)",
"exdate": [],
"dtstamp": "2024-01-15T01:48:11.000Z",
"uid": "[email protected]",
"created": "2024-01-08T22:47:14.000Z",
"description": "<anonymized description>",
"lastmodified": "2024-01-08T22:47:14.000Z",
"sequence": "0",
"status": "CONFIRMED",
"summary": "<anonymized - this was a meeting name>",
"transparency": "OPAQUE",
"method": "PUBLISH"
} |
Also, I see this error show up ~460 times in the same console
not sure if that is relevant |
Thanks!
|
I use google calendar. And its showing it on the wrong day - specifically a day earlier. The event is supposed to be tomorrow evening, but this plugin is showing me that it is taking place today. Here is the raw VEVENT event from .ics file
As you can see, FREQ=WEEKLY and WKST=SU and BYDAY=MO. The week should be starting on Sunday, and it should not be showing me this event tomorrow (monday). But the cloned event object (from my previous message above) is suggesting my "wkst" is Saturday (6), and "byweekday" as Sunday (0). Could that be the issue? |
There might be an issue with |
@devangnegandhi - can you test with https://github.com/cloud-atlas-ai/obsidian-ics/releases/tag/1.6.1test ? Specifically replace |
All the massive # of console errors are no longer there. BUT unfortunately, the issue still persists. Here is an updated output in the console based on the new main.js file you provided
{
"tzid": "America/Los_Angeles",
"dtstart": "2024-01-09T04:00:00.000Z",
"freq": 2,
"wkst": {
"weekday": 6
},
"until": "2024-03-06T07:59:59.000Z",
"byweekday": [
{
"weekday": 0
}
]
}
{
"type": "VEVENT",
"params": [],
"start": "Sun Jan 14 2024 20:00:00 GMT-0800 (Pacific Standard Time)",
"datetype": "date-time",
"end": "Sun Jan 14 2024 21:00:00 GMT-0800 (Pacific Standard Time)",
"exdate": [],
"dtstamp": "2024-01-15T02:37:59.000Z",
"uid": "[email protected]",
"created": "2024-01-08T22:47:14.000Z",
"description": "<anonymized description>",
"lastmodified": "2024-01-08T22:47:14.000Z",
"sequence": "0",
"status": "CONFIRMED",
"summary": "<anonymized - this was a meeting name>",
"transparency": "OPAQUE",
"method": "PUBLISH"
} |
@muness here is a minmal .ics file that has 1 event that I can consistently reproduce this issue with. It only returns an event on Sundays (eg: 2024-01-14), whereas it is supposed to recur on Mondays (eg 2024-01-15). Can you confirm if you can reproduce this issue as well?
|
Interestingly, for me, in the Eastern timezone this does show up on Mondays. Let's see what happens when I switch my machine to PT... |
I don't believe so. From my understanding of the spec and also ran by ChatGPT:
|
When I switch to PT, the event moves to Sunday, as @devangnegandhi is experiencing. |
I'm somewhat relieved that I'm not the only one experiencing this issue. I'm relieved that @devangnegandhi has been able to provide better feedback that I have been on this. Just an FYI, I am also PT, and I recently had to change my calendar view both in Obsidian and in Google calendar for weeks to start on Mondays because the file.day.weekyear call only uses isoWeeks. I'll provide feedback on behavior for upcoming reoccuring events. Thanks again for contuning to chase this down. |
This appears to be a symptom of jkbrzt/rrule#452. Specifically:
Ooph... I'll see if I can apply the guidance in the documentation:
|
I don't think I tried switching my own timezone when I tried reproducing the issue previously. That seems to be enough of an offset to trigger the issue in these examples. Now that I think of it, I have seen the issue in my own calendar, for an all day No Meetings Day event when daylight savings changed. |
@devangnegandhi , @dlhawe - please give https://github.com/cloud-atlas-ai/obsidian-ics/releases/tag/1.6.2-rrule-utc-coercion a try. In my testing, I see a new issue where some events show up on multiple days, but with the correct date. e.g.
is showing up for me under both the 2024-01-23 and 2024-01-24 journal entries. Let me know if you see other issues. Thanks! |
Did I mention lately how much fun timezones are?! |
@muness This is showing me the event for today correctly
But, I am seeing this issue as well - an event that already happened yesterday is showing up for me today
|
Let me keep an eye on my events this week, and I will keep you updated on what I see |
Thank you! I think this comes down to the way the timezoned events are matching to non-timezoned matching now. I can add a second filter to make sure they're in the right day. I am trying to think of a better way to match events in the first place. Maybe something like converting everything to UTC first then back. Or maybe I can find another calendar client to see how they address this. |
Thanks for the patience @dlhawe , @devangnegandhi , it's been a crazy few weeks at home and work. I think I figured out a way to address the remaining bug in the fix. Can one of you try https://github.com/cloud-atlas-ai/obsidian-ics/releases/tag/1.6.2-beta2 please? Thanks so much! |
Hi @muness - unfortunately, this changed has caused a regression. I am seeing a recurring event that is supposed to be tomorrow showing up for me today. Also, a recurring event that is supposed to be today is not there - it shows up if I fetch the events for yesterday. |
BTW, your previous change was working fine for most of the event - except for some that were showing up for a day later. |
Well, that's unexpected. It sounds like the event itself (in the ics) must be in UTC sometimes, and therefore the last change makes it no match. I am beginning to wonder if I should just match from the start of the previous day through the next day and then filter events down to just those on the day we're matching. Sigh... I'll put together a test build tonight. |
k, one more attempt, with the multi-day UTC check for rrule events then narrowing back down to just today in the current timezone. https://github.com/cloud-atlas-ai/obsidian-ics/releases/tag/1.6.2-beta3 Please let me know if this fixes things, @devangnegandhi |
Closing with #113. Please reopen with details if you still see this issue. 🙏 |
I'm using the ICS plugin to pull events from my Google calendar into my daily notes. I am having an issue where a shared event (where I am a guest) is showing up a day early in my daily notes, but calendar events I enter are showing up on the proper day. There doesn't appear to be anything in the configuration settings to explain thus behavior.
The text was updated successfully, but these errors were encountered: