mattermost / mattermost-plugin-google-calendar

Mattermost Google Calendar Plugin
36 stars 35 forks source link

Location field missbehavior #78

Closed fmartingr closed 2 months ago

fmartingr commented 5 months ago

Investigate the report of this behaviour using the create event modal:

Information entered in the "Location" field in the Create Calendar Event window in MM is not transferring to the location value in Google Calendar and not showing in the 5-minute reminder.

But entering Location info in the "Description" field will transfer to the Location field in Google Calendar and show up in the 5-minute reminder.

Kshitij-Katiyar commented 5 months ago

@fmartingr Can you please elaborate on the issue and maybe share some video about this? Below I have created the event through the Google Calendar plugin and included the location in the location field only. The same was reflected on the Google side as well I got the reminder as well. Below are some screenshots:

Screenshot from 2024-05-31 18-57-01 WhatsApp Image 2024-05-31 at 18 57 27

Can you please share the steps to reproduce this maybe or any other additional information is appreciated. cc: @mickmister

mickmister commented 5 months ago

But entering Location info in the "Description" field will transfer to the Location field in Google Calendar and show up in the 5-minute reminder.

@Kshitij-Katiyar I think the 5-minute reminder is referring to the Mattermost plugin's DM reminder

Kshitij-Katiyar commented 5 months ago

But entering Location info in the "Description" field will transfer to the Location field in Google Calendar and show up in the 5-minute reminder.

@Kshitij-Katiyar I think the 5-minute reminder is referring to the Mattermost plugin's DM reminder

sure @mickmister I didn't know about that. Let me check that but still, the location field is correctly getting configured on the google side.

fmartingr commented 5 months ago

I will gather more information and get back to you @Kshitij-Katiyar.

mickmister commented 5 months ago

It seems we're using the event's location in the reminder post so not sure where the issue is occurring. @fmartingr Do you know if this is specifically for a push notification, or the general post that's created?

https://github.com/mattermost/mattermost-plugin-mscalendar/blob/417dbb0baa1be3ef85bd4951f5795cf5fddbc2d4/calendar/engine/views/calendar.go#L152

Kshitij-Katiyar commented 4 months ago

I will gather more information and get back to you @Kshitij-Katiyar.

@fmartingr Is there any update on this?

fmartingr commented 4 months ago

I will gather more information and get back to you @Kshitij-Katiyar.

@fmartingr Is there any update on this?

We have requested a video from the customer to understand what is the problem they are experiencing. Still waiting on it.

wiggin77 commented 4 months ago

We have requested a video from the customer to understand what is the problem they are experiencing.

Does this mean we are unable to reproduce this? @fmartingr

fmartingr commented 4 months ago

We have requested a video from the customer to understand what is the problem they are experiencing.

Does this mean we are unable to reproduce this? @fmartingr

What we think is that is not that there's a problem with the field itself since it's saving correctly, but there's a problem displaying the field in some part of the UI.

darias416 commented 2 months ago

@fmartingr the customer confirmed this issue has been resolved for them as of version 1.1.1

cc: @wiggin77