search

mattermost-community / mattermost-plugin-agenda

Mattermost plugin to handle meeting agendas
Apache License 2.0
35 stars 21 forks source link

Agenda list command does not work on mobile #50

Open DHaussermann opened 4 years ago

DHaussermann commented 4 years ago

The design of Agenda for showing list in the RHS work well in webapp but it does not accommodate the mobile app.

When you run /agenda list on mobile nothing seems to happens

Steps:

Observed: Nothing happened Expected: Not sure. If the behavior is consistent with other hashtag searches, I suppose it should open some search results for the appropriate hashtag.

marianunez commented 4 years ago

The /agenda list command is tightly coupled with the UI and unfortunately plugins UI actions are currently not supported in mobile. I would consider this an enhancement more than a bug.

hanzei commented 4 years ago

/agenda list should show a ephemeral message with all the agenda items on mobile.

marianunez commented 4 years ago

/agenda list should show a ephemeral message with all the agenda items on mobile.

@hanzei Do we want this ephemeral message also in the webapp that already opens the RHS? How will the plugin know If the command is from mobile to act different?

hanzei commented 4 years ago

According to @larkox it's possible to detect the type of client that is used to trigger a slash command. It should just be a normal ephemeral post.

larkox commented 4 years ago

The way to access this is checking args.Session.IsMobileApp(). Nevertheless, apparently the Session part is to be deprecated eventually (v6.0) so not sure if we should use it or not. If we were not to use that, not sure how we can access this information (getting the session from the session ID in the context maybe?).