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

fix(cs3-message-overlap): add changes ben suggested #799

Merged
merged 3 commits into from
Oct 4, 2024
Merged

Conversation

jdinh8124
Copy link
Collaborator

Purpose

Adjust an issue where error messages were overlapping in fieldArrays

Linked Issues to Close

https://qmacbis.atlassian.net/browse/OY2-30237

@jdinh8124 jdinh8124 self-assigned this Sep 26, 2024
@jdinh8124 jdinh8124 added status: READY PR is ready for review status: TESTING PR is being tested by QA labels Sep 26, 2024
Copy link
Contributor

github-actions bot commented Sep 26, 2024

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 62.08% 4509 / 7263
🔵 Statements 61.7% 4776 / 7740
🔵 Functions 54.45% 1234 / 2266
🔵 Branches 33.3% 857 / 2573
File CoverageNo changed files found.
Generated in workflow #332 for commit 95160d1 by the Vitest Coverage Report Action

Copy link

codeclimate bot commented Oct 2, 2024

Code Climate has analyzed commit 95160d1 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 62.0% (0.0% change).

View more on Code Climate.

@jdinh8124 jdinh8124 merged commit dcfe7b3 into main Oct 4, 2024
16 checks passed
@jdinh8124 jdinh8124 deleted the overlap-valid branch October 4, 2024 18:16
Copy link
Contributor

🎉 This PR is included in version 1.5.0-val.87 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released on @val status: READY PR is ready for review status: TESTING PR is being tested by QA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants