-
Notifications
You must be signed in to change notification settings - Fork 112
feat(hub): add fix and verify html tests #20617
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
1bb6667 to
6eeffb3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should not be fixed this way, as we are saying all cells are headers which is not good for accessibility
We should consider reworking this part to not use a table (since it has no header)
I let you free to either fix it in this PR or create a dedicated ticket (in which we could include the support ticket table as well)
a796169
a796169 to
7263a51
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The name of this file should be BillingStatus.spec.tsx
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
ref: #MANAGER-20181 Signed-off-by: achraf hafedh <[email protected]>
c8fc007 to
86c98ee
Compare
ref: #MANAGER-20181
Description
Ticket Reference: #MANAGER-20181
Additional Information