search

mattermost / mattermost-plugin-apps

Powers the Mattermost App Framework
https://developers.mattermost.com/integrate/apps/
Apache License 2.0
33 stars 29 forks source link

[MM-49946] App Marketplace installs should always be triggered by /apps install listed #441

Closed hanzei closed 1 year ago

hanzei commented 1 year ago

Summary

Simplify the UX by triggering Marketplace always installs via /apps install listed. This makes the documentation easier to write as there are no different cases to handle.

With HTTP apps disabled

Screenshot from 2023-01-30 15-53-33

With HTTP apps enabled

Screenshot from 2023-01-30 15-53-50

Ticket Link

https://mattermost.atlassian.net/browse/MM-49946

codecov-commenter commented 1 year ago

Codecov Report

Base: 20.69% // Head: 20.69% // No change to project coverage :thumbsup:

Coverage data is based on head (7184183) compared to base (f253391). Patch has no changes to coverable lines.

:exclamation: Current head 7184183 differs from pull request most recent head 9a20026. Consider uploading reports for the commit 9a20026 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #441 +/- ## ======================================= Coverage 20.69% 20.69% ======================================= Files 80 80 Lines 5319 5319 ======================================= Hits 1101 1101 Misses 4089 4089 Partials 129 129 ``` Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=mattermost). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=mattermost)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.