Mattermost-server: Add bool (boolean) type for Interactive Dialogs

Created on 18 Mar 2019  路  8Comments  路  Source: mattermost/mattermost-server

If you're interested please comment here and come join our "Contributors" community channel on our daily build server, where you can discuss questions with community members and the Mattermost core team. For technical advice or questions, please join our "Developers" community channel.

New contributors please see our Developer's Guide.


Notes: Jira ticket

Interactive Dialogs allow integrations to receive some form-styled input from users and have that input submitted back to the integration. An Interactive Dialog consists of a list of elements. These elements have a defined type such as text or select.

This ticket is about adding a bool type element for Dialog elements. Setting this type for an element should render a checkbox like:

[X] The meeting was helpful.

For an example UI style containing a checkbox, see Account Settings > Notifications > Words that trigger mentions

The default value is false. The integration might change this by setting default' totrue`.

This ticket requires changes in both the web app and the mobile app.

For more information about Interactive Dialog, see https://docs.mattermost.com/developer/interactive-dialogs.html

If you have any questions or would like feedback, feel free to ask in the Developer Toolkit channel on our Mattermost community instance.

AreIntegrations Medium Good First Issue Hacktoberfest Help Wanted PR Exists TecReact Native TecReactJS

All 8 comments

working on it

Thanks @hvhallmann. Let me know, if you have questions.

Webapp is merged. Now only mobile is missing.

Hey @hvhallmann,

Just want to sync up with you: Are you still interested in adding this feature for the mobile app?

Hey @hanzei,
I'm sorry, but I don't think I will have time in the next couple of weeks

All fine! Thanks for the heads up and all you spend on the awesome feature :+1:

Hi @hanzei,
Can you confirm if the issue is Up For Grabs? I noticed a merged PR to address it -https://github.com/mattermost/mattermost-mobile/pull/3239.

Good catch @marianacapelo :+1:. This was competed with https://github.com/mattermost/mattermost-mobile/pull/3239. Closing

Was this page helpful?
0 / 5 - 0 ratings