-
Notifications
You must be signed in to change notification settings - Fork 243
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
Show upcoming events for the current conversation inside Talk #5909
Comments
How does it supposed to look on the Talk side? Just a conversation description or something more complex, e.g., new UI element or sidebar's tab on web/desktop? |
Maybe it could work like the OOO for events that are on the same or next day? It shouldn't be hidden somewhere deep in the sidebar since that defeats the purpose IMO. |
Then it's not defined what to do if there is OOO already. And that if there are several meetings planed or user want to know about the meeting in advance? |
Having and OOO status but also a scheduled meeting is unlikely, but still should be accounted for. |
I'm also not sure I understand how conversations and events get connected. Can I have many events connected to one conversation? Can I connect any existing event to a conversation, or it's only for conversations created from events? |
The connection is a simple value match. Events that have the talk conversation set as LOCATION will count towards upcoming events. This could be an event you created for yourself, one in a calendar shared with you or an event where you were invited (team meeting). Talk room to event relation is 1:n. Multiple events can have the same URL as location. |
A PoC can bee seen at nextcloud/server#45435. |
In this case I'd vote for a new Tab in the sidebar to have an ability to see all the events. Though, we always complain about the number of tabs making them small and unreadable... |
I think in a tab it would be hidden too much. How about the header bar? This could be a popover menu that shows the x next meetings when clicked @marcoambrosini do you have any other ideas? |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
What about having a setting for this? Per room and a global (default) one? Something along one of these:
That are just a few ideas I was able to come up with so far 😅
To hook on this one: I'd actually think it would be more intuitive to have the banner act as a link to the calendar when clicked. Ideally without "closing" i.e. leaving Talk, but opening in a new tab. It could also just open the list in the sidebar (if implemented) and show the event details there. |
Hi @MPStudyly, we generally add a new setting only if absolutely necessary. I think that here if there is an upcoming meeting, it's good to show it on the day of the meeting. And if you dismiss it that 's it, we don't show it anymore for that meeting :)
Yes that's the idea :) |
While I understand your reasoning and second the idea to not clutter users with loads of settings, I wouldn't underestimate the need to customize even such details. Unfortunately I don't have a good idea (yet) on how to keep the general settings page as clean as possible while still allowing for such changes. Still, I know from my own experience in our company that a lot of users would love to tweak some minor stuff to their liking, especially around all sorts of notifications 😅 Don't let this stall the implementation though. I think having this feature to begin with is already a good start. Settings can always be added in later :)
Great to hear! Unexpectedly leaving the page (or app in this case) is always somewhat annoying 😅 |
@nickvergessen & @ChristophWurst ping me when this gets picked up :) |
Assigning @DorraJaouad for Talk frontend. @jancborchardt will create a new screenshot of the last design idea how to put it in the header for now. |
@jancborchardt do you want to take over and assign to you then? |
Here’s what I’m thinking for the web interface, very similar to what @ChristophWurst proposed.
On mobile it could go below the header bar. Much like @marcoambrosini’s mockup, but docked to the header instead of floating, not blue, and not dismissable. |
@DorraJaouad do you have any questions regarding the mockup or does it make sense? :) |
Is it by opening in a new tab to Calendar app (e.g: with event on the side bar in month view, but need to check if API could provide a link in payload) or we can make a modal showing details of the event so user can stay in Talk? |
New tab (to reduce foreign code hosting and so desktop client is supported as well) |
How far should we go?
I'd still expose it in the API so we can include it in the future if this changes. I think the meeting title is only identical with very specific channels. E.g. in the design channel I'd appreciate to see if there is a weekly design team call or Mail design review. |
I would say 4 weeks or 31 days? Or maybe even API parameter? |
Yes, it would be cool if it’s something of that sort so monthly calls are covered.
Yup sounds good, then we can adjust later on. |
Is your feature request related to a problem? Please describe.
As a Nextcloud Calendar and Talk user I often schedule meetings for individuals and groups. The meetings happen in Talk because we are a remote company.
If I am in Calendar, I can see upcoming events and where they take place.
If I am in Talk I can not see any of this information.
Describe the solution you'd like
When I open a conversation, check my calendar(s) for any events in the next x hours/days. Then list everything where the current conversation URL matches the LOCATION/URL of any of the events.
Work packages
Additional context
N/a
The text was updated successfully, but these errors were encountered: