Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

ABC-193: Add webapp commit hash to "about" modal #751

Merged
merged 2 commits into from
Feb 9, 2018

Conversation

ccbrown
Copy link
Contributor

@ccbrown ccbrown commented Feb 8, 2018

Summary

screen shot 2018-02-07 at 6 17 10 pm

Ticket Link

https://mattermost.atlassian.net/browse/ABC-193

Checklist

  • Ran make check-style to check for style errors (required for all pull requests)
  • Has UI changes
  • Includes text changes and localization file (.../i18n/en.json) updates

@ccbrown ccbrown added 1: PM Review Requires review by a product manager 2: Dev Review Requires review by a core commiter labels Feb 8, 2018
@jasonblais jasonblais added Setup Old Test Server Triggers the creation of a test server and removed Setup Old Test Server Triggers the creation of a test server labels Feb 8, 2018
@jasonblais jasonblais self-assigned this Feb 8, 2018
@mattermost mattermost deleted a comment from mattermod Feb 8, 2018
@mattermost mattermost deleted a comment from mattermod Feb 8, 2018
@mattermost mattermost deleted a comment from mattermod Feb 8, 2018
@jasonblais jasonblais added the Setup Old Test Server Triggers the creation of a test server label Feb 8, 2018
@jasonblais
Copy link
Contributor

@ccbrown Looks like the build failed,

@ccbrown
Copy link
Contributor Author

ccbrown commented Feb 8, 2018

@jasonblais fixed.

@jasonblais jasonblais removed the Setup Old Test Server Triggers the creation of a test server label Feb 8, 2018
@mattermost mattermost deleted a comment from mattermod Feb 8, 2018
@mattermost mattermost deleted a comment from mattermod Feb 8, 2018
@mattermost mattermost deleted a comment from mattermod Feb 8, 2018
@jasonblais jasonblais added the Setup Old Test Server Triggers the creation of a test server label Feb 8, 2018
@jasonblais
Copy link
Contributor

Tested at different screen widths and zoom levels.

@ccbrown Only question is whether it's obvious the Build Hash and EE Build Hash refer to the server and enterprise branches?

@ccbrown
Copy link
Contributor Author

ccbrown commented Feb 8, 2018

@jasonblais 🤷‍♂️ I think it's at least more obvious than it was before now that the webapp hash is also there. Being labeled "webapp", it kinda implies that the others aren't webapp.

Copy link
Contributor

@jasonblais jasonblais left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fair enough. We can always update if someone is confused.

@jasonblais jasonblais removed 1: PM Review Requires review by a product manager Setup Old Test Server Triggers the creation of a test server labels Feb 8, 2018
@jasonblais jasonblais removed their assignment Feb 8, 2018
@mattermost mattermost deleted a comment from mattermod Feb 8, 2018
@mattermost mattermost deleted a comment from mattermod Feb 8, 2018
@mattermost mattermost deleted a comment from mattermod Feb 8, 2018
@ccbrown ccbrown merged commit cbecb33 into mattermost:master Feb 9, 2018
@lindalumitchell lindalumitchell added the Tests/Not Needed Does not require new release tests label Feb 14, 2018
@amyblais amyblais added Changelog/Done Required changelog entry has been written Docs/Not Needed Does not require documentation labels Feb 17, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2: Dev Review Requires review by a core commiter Changelog/Done Required changelog entry has been written Docs/Not Needed Does not require documentation Tests/Not Needed Does not require new release tests
Projects
None yet
6 participants