mattermost-community / mattermost-plugin-autotranslate

Apache License 2.0
41 stars 19 forks source link

Fix and updated the plugin #27

Closed saturninoabril closed 4 years ago

saturninoabril commented 4 years ago

Summary

Test steps:

  1. On initial run, autotranslate plugin is disabled to a user. Post a message and check that “Translate” is not available in post’s dropdown menu
  2. See ephemeral help by posting /autotranslate or /autotranslate help. Check text, words or links for any discrepancy.
  3. Enable the plugin for current user by /autotranslate on and see that it’s enabled with source: Auto and target: English.
  4. Post any message in a different language supported by AWS Translate
    • Check that “Translate” is available in that post. (Note: “Translate” is available to regular post only. Other post like system message should not have that option.)
    • Click “Translate” and see that the post is translated.
    • Click “close” to hide the translated message
  5. Try to change different settings (info, source, target, of/off) and see if it works or translates as expected.
saturninoabril commented 4 years ago

(Hmm, diff to go.sum is a bit annoying. Not happening on local. btw, I'll update all at once after dev review)

saturninoabril commented 4 years ago

Thanks Jesse and Ben for the reviews. PR now updated, PTAL.

saturninoabril commented 4 years ago

Thank you for testing Ogi!