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

Update to v1.1.0 makes it disappear button "Create Talk room for this event" in Calendar event creation #92

Closed
fabiorasta opened this issue Jun 21, 2024 · 3 comments · Fixed by nextcloud/calendar#6103
Labels
bug Something isn't working

Comments

@fabiorasta
Copy link

Which version of assistant are you using?

1.0.9

Which version of Nextcloud are you using?

29.0.2

Which browser are you using? In case you are using the phone App, specify the Android or iOS version and device please.

Firefox 127

Describe the Bug

After updating to version 1.1.0 of the Assistant AI app, I noticed that, when creating a new calendar event (in the Calendar app), the "Create Talk room for this event" button was no longer available (in the section where the user invites participants). By reinstalling the previous version, i.e. 1.0.9, everything goes back to normal.

Expected Behavior

I expect that with the update to version 1.1.0 of the Assistant AI app, the creation of a new calendar event (in the Calendar app) will not be affected, i.e. that the "Create Talk room for this event" button will remain available

To Reproduce

Update to version 1.1.0 the Assistant AI app.
Create a new calendar event (in the Calendar app).
You will notice that the "Create Talk room for this event" button was no longer available (in the section where the user invites participants).

@fabiorasta fabiorasta added the bug Something isn't working label Jun 21, 2024
@julien-nc
Copy link
Member

Thanks for reporting this.

So, Calendar is getting a wrong information about Talk being enabled or not.
I narrowed it down to this:

https://github.com/nextcloud/assistant/blob/main/lib/Listener/BeforeTemplateRenderedListener.php#L48

Without this line everything is fine.
Weird thing is that when checking which value is provided to the frontend, it is always correct there:
https://github.com/nextcloud/calendar/blob/main/lib/Controller/ViewController.php#L121
Then I found this:

https://github.com/nextcloud/calendar/blob/main/lib/Listener/CalendarReferenceListener.php#L96

The assistant is dispatching the RenderReferenceEvent event because the chattyUI might need to resolve and render link previews.
But when Calendar receives RenderReferenceEvent, it wants its frontend to consider that Talk is disabled (probably just to avoid having the "create talk room" button in link previews).
So Calendar considers that RenderReferenceEvent is never dispatched when loading the main Calendar app page.

I think this is a Calendar issue. Wdyt @miaulalala @hamza221 ?

@hamza221
Copy link

hamza221 commented Jul 2, 2024

Then I found this:

https://github.com/nextcloud/calendar/blob/main/lib/Listener/CalendarReferenceListener.php#L96

This was implemented for widget creation in (talk, text ...)
It is indeed a Calendar issue , feel free to transfer it to the calendar repo, I'll take over.

@tcitworld
Copy link
Member

Related to nextcloud/calendar#6076

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants