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

Updated Accessibility tests based on the latest label changes on the App #5249

Merged
merged 1 commit into from
Apr 2, 2020
Merged

Updated Accessibility tests based on the latest label changes on the App #5249

merged 1 commit into from
Apr 2, 2020

Conversation

srkgupta
Copy link
Contributor

@srkgupta srkgupta commented Apr 1, 2020

Summary

Updated failing Accessibility tests based on the latest label changes on the App

Ticket Link

N/A. Updated based on daily runs.

@srkgupta srkgupta added the 3: QA Review Requires review by a QA tester label Apr 1, 2020
josephbaylon
josephbaylon approved these changes Apr 1, 2020
View reviewed changes
Copy link
Contributor

@josephbaylon josephbaylon left a comment

Choose a reason for hiding this comment

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

LGTM. All tests passed.

prapti
prapti approved these changes Apr 2, 2020
View reviewed changes
Copy link
Contributor

@prapti prapti left a comment

Choose a reason for hiding this comment

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

👍

@prapti prapti added QA Review Done and removed 3: QA Review Requires review by a QA tester labels Apr 2, 2020
@prapti prapti merged commit 09014d4 into mattermost:master Apr 2, 2020
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Apr 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@josephbaylon josephbaylon josephbaylon approved these changes

@prapti prapti prapti approved these changes

Assignees
No one assigned
Labels
Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation QA Review Done
Projects
None yet
Milestone
No milestone
4 participants
@srkgupta @josephbaylon @prapti @amyblais