Skip to content
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

[TB 92 and 37]comments_async: update semantics and implement some fixes from a11y test #1490

Merged
merged 3 commits into from
Sep 21, 2023

Conversation

philli-m
Copy link
Contributor

@philli-m philli-m commented Sep 7, 2023

Some fixes for the tech backlog stories concerning the comments, has been tested with a+ main, may need additional fixes when mB is updated but can be done in design sprint

@github-actions
Copy link

github-actions bot commented Sep 7, 2023

Coverage report

Total coverage

Status Category Percentage Covered / Total
🔴 Statements 10.98% 120/1093
🔴 Branches 12.95% 89/687
🔴 Functions 11.11% 39/351
🔴 Lines 18.56% 647/3486

Status of coverage: 🟢 - ok, 🟡 - slightly more than threshold, 🔴 - under the threshold

Report generated by 🧪jest coverage report action from 31f348e

@philli-m philli-m changed the title comments_async: update semantics and implement fixes from a11y test WIP comments_async: update semantics and implement fixes from a11y test Sep 7, 2023
@philli-m philli-m force-pushed the pm-2023-09-comment-a11y branch 3 times, most recently from 98105a0 to 0de72e8 Compare September 11, 2023 11:02
@philli-m philli-m changed the title WIP comments_async: update semantics and implement fixes from a11y test comments_async: update semantics and implement fixes from a11y test Sep 11, 2023
@philli-m philli-m changed the title comments_async: update semantics and implement fixes from a11y test [TB 92 and 37]comments_async: update semantics and implement fixes from a11y test Sep 11, 2023
@philli-m philli-m changed the title [TB 92 and 37]comments_async: update semantics and implement fixes from a11y test [TB 92 and 37]comments_async: update semantics and implement some fixes from a11y test Sep 11, 2023
Copy link
Contributor

@goapunk goapunk left a comment

Choose a reason for hiding this comment

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

looks good to me!

@goapunk goapunk merged commit 8710f0d into main Sep 21, 2023
2 checks passed
@goapunk goapunk deleted the pm-2023-09-comment-a11y branch September 21, 2023 10:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants