Fix System Console items not appearing for appropriate license levels

Description

Additional System Console issues found, fixes to be included in v5.13.2:

1. Environment > File Storage > Enable Server-Side Encryption for Amazon S3 missing from TE

2. Environment > Push Notification Server > Checkbox setting for accepting Mattermost HPNS terms is missing from E10 & E20 (“I understand and accept Mattermost HPNS ToS & PP... (checkbox)”)

  • Should be hidden on TE (because Mattermost HPNS is not an option in the dropdown)

  • Should be present on both E10 and E20

  • CORRECTION: Confirmed this is present in E10 & E20

3. Environment > High Availability > Read only Config missing from E20

  • Should be hidden on TE and E10

  • Should be present E20 only

4. Site Configuration > Notifications > Email Notification Contents missing from E10

5. Site Configuration > File Sharing & Downloads > Allow File Uploads on Mobile missing from E10

6. Site Configuration > File Sharing & Downloads > Allow File Downloads on Mobile missing from E10

7. OAuth 2.0 > Google

  • Should be hidden in E10 & TE

  • Should be visible in only E20

8. OAuth 2.0 > Office 365

  • Should be hidden in E10 & TE

  • Should be visible in only E20

QA Test Steps

See description for list of items to check (check all license levels for each one).

Have on hand an E10 and an E20 license. Remove license to test TE, then add E10 license, then remove and add E20 license.

Activity

Show:
Katie Wiersgalla
July 22, 2019, 8:26 PM

Regarding 2. I confirmed this is working as expected on E10.

Katie Wiersgalla
July 22, 2019, 8:27 PM

Per conversation with Martin - appears to be a bug for showing/hiding 7 & 8 in code - these are valid issues.

Linda Mitchell
July 23, 2019, 12:40 AM

Thank you both for sorting all of these out!

Linda Mitchell
July 24, 2019, 4:29 AM

On 5.13.2-rc1 (prev.test.mattermost.com), I'm still not seeing 3) Environment > High Availability > Read only Config on E20. Checking with Martin and Katie in a GM if I'm understanding that one correctly (and/or if it is dependent on another setting).

Went through spreadsheet with all three license levels, and did not find any other discrepancies (including the above except for #3).

Linda Mitchell
July 24, 2019, 1:48 PM

Verified with Martin and Katie that item 3 is invalid; that setting was removed: https://github.com/mattermost/mattermost-webapp/pull/2391

No other issues were found; closing. Added to License Levels tab in spreadsheet and labeled PR.

Done

Mana

None

Assignee

Martin Kraft

QA Assignee

Linda Mitchell

Reporter

Linda Mitchell

Epic Link

None

Fix versions

Mattermost Team

Sustained Engineering

Sprint

None

Labels

None

QA Testing Areas

System Console

GitHub Issue

None

Components

None

Severity

None
Configure