Skip to content
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

Calendar widget is empty after update to 22.2.2 #3649

Closed
jakobroehrl opened this issue Nov 13, 2021 · 21 comments · Fixed by #3682
Closed

Calendar widget is empty after update to 22.2.2 #3649

jakobroehrl opened this issue Nov 13, 2021 · 21 comments · Fixed by #3682
Assignees
Labels
3. to review Waiting for reviews bug regression Regression of a previous working feature

Comments

@jakobroehrl
Copy link
Contributor

jakobroehrl commented Nov 13, 2021

calendar: 2.3.4

same with: 23.0.0 RC2 and v2.4.0 RC1

image

@szaimen szaimen transferred this issue from nextcloud/server Nov 13, 2021
@flatterrich
Copy link

the same here

@tcitworld tcitworld added 1. to develop Accepted and waiting to be taken care of bug labels Nov 15, 2021
@tcitworld tcitworld added this to the v2.3.5 milestone Nov 15, 2021
@ChristophWurst
Copy link
Member

I can reproduce with one of the two productive instances that I have access to. So this bug seems to depend on some event data.

@ChristophWurst
Copy link
Member

ChristophWurst commented Nov 15, 2021

Bildschirmfoto von 2021-11-15 11-06-59

So for me it's this code

try {
const calendarObject = mapCDavObjectToCalendarObject(r, calendar.id)
calendarObjects.push(calendarObject)
calendarObjectIds.push(calendarObject.id)
} catch (e) {
console.error('could not convert calendar object', e)
}

It is not the Deck calendars/events. Disabling them still triggers the error.

@ChristophWurst
Copy link
Member

With #3651 we should gain more insights. Will tag an RC2 once it's in and then we can try again 🙏

@ChristophWurst ChristophWurst added 2. developing Work in progress and removed 1. to develop Accepted and waiting to be taken care of labels Nov 15, 2021
@ChristophWurst ChristophWurst self-assigned this Nov 15, 2021
@ChristophWurst ChristophWurst added the regression Regression of a previous working feature label Nov 15, 2021
@ChristophWurst
Copy link
Member

For me it's all the todo items in my calendar. Redacted example:

BEGIN:VCALENDAR
VERSION:2.0
PRODID:+//IDN bitfire.at//ical4android
BEGIN:VTODO
DTSTAMP:20190825T082453Z
UID:494fbd55-4c0c-4e56-bffa-39c1df8c8b0c
SEQUENCE:1
CREATED:20190825T073208Z
LAST-MODIFIED:20190825T082416Z
SUMMARY:redacted
STATUS:IN-PROCESS
END:VTODO
END:VCALENDAR

@tcitworld
Copy link
Member

I have the issue with all kind of items : recurring events, birthday calendars, todos, simple events.

@tcitworld
Copy link
Member

Note : the Tasks dashboard widget works properly with the same set of todos.

@jakobroehrl
Copy link
Contributor Author

With RC2:
image

@ChristophWurst
Copy link
Member

if you expand the {response: n} then you'll see the event/task data causing this

@jakobroehrl
Copy link
Contributor Author

That are the deck cards:
image

But if I disable deck it's also not showing any data in the calendar widget

@ChristophWurst
Copy link
Member

Bildschirmfoto von 2021-11-15 20-12-17

it throws right there inside calendar-js. but the argument is a property as far as I can tell. this could be an issue with the build, and therefore not show locally with the dev/watch bundles. I'll keep digging.

@svankan
Copy link

svankan commented Nov 15, 2021

I have the issue with all kind of items : recurring events, birthday calendars, todos, simple events.

I have the same problem after upgrading to Nextcloud version: 21.0.6.2 and calendar: 2.3.4.

@larwood
Copy link

larwood commented Nov 17, 2021

I have not yet updated and am still on 22.2.0 but I did update the Calendar App to 2.3.4 this morning and the widget is now broken for me as well.

@Nonlinearsound
Copy link

Nonlinearsound commented Nov 17, 2021

Christoph, same error here. On my instance the logger shows me that there are two events being loaded, which is right a my calendar only holds two elements. I attached two ics files for you to this reply. Maybe you'll find some attribute that is not formatted right?

DA454A53-C990-4EC3-AF6A-E99F054CAD8E.ics.txt

9DF693D8-0E7F-472F-A0DC-E34CBBBAD21A.ics.txt

Edit: It only effects the Dashboard widget.

@dcommisso
Copy link

Same problem here. Also, mobile calendar no longer synchronizes with nextcloud calendar, maybe it's related?

Nextcloud version: 22.2.3
calendar version: 2.3.4

@blizzz
Copy link
Member

blizzz commented Nov 17, 2021

Same problem here. Also, mobile calendar no longer synchronizes with nextcloud calendar, maybe it's related?

Nextcloud version: 22.2.3 calendar version: 2.3.4

I cannot confirm the calendar sync issue, this continues to work well for me.

@flatterrich
Copy link

for me too, only widget without function

@jakobroehrl
Copy link
Contributor Author

Sync is working without any problems and calendar, too.
Only the dashboard widget is the problem on my installation.

@dcommisso
Copy link

Same problem here. Also, mobile calendar no longer synchronizes with nextcloud calendar, maybe it's related?

Nextcloud version: 22.2.3 calendar version: 2.3.4

Never mind, it turns out that there was a problem with calendar mobile app. I apologize.

@larwood
Copy link

larwood commented Nov 26, 2021

This issue is now resolved for me after updating to v2.4.0 on NC 22.2.3

@bcutter
Copy link

bcutter commented Nov 26, 2021

Yes I can confirm. Even this bug is not listed in the fix section of the release notes, it's now finally working again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug regression Regression of a previous working feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.