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(g2 divider fix): fix divider issues on g2a and g2b #728

Merged
merged 5 commits into from
Sep 6, 2024
Merged

Conversation

jdinh8124
Copy link
Collaborator

Purpose

There was an issue where a few dividers were missing on g2a and g2b. We added logic to include these divider bars in sections that the figma required.

Linked Issues to Close

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

Copy link
Contributor

github-actions bot commented Aug 21, 2024

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 54.46% 4273 / 7845
🔵 Statements 54.28% 4497 / 8284
🔵 Functions 48.46% 1023 / 2111
🔵 Branches 27.06% 801 / 2959
File CoverageNo changed files found.
Generated in workflow #157

@jdinh8124 jdinh8124 marked this pull request as ready for review August 22, 2024 18:53
Copy link

codeclimate bot commented Aug 29, 2024

Code Climate has analyzed commit fb17aae 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 54.4% (0.0% change).

View more on Code Climate.

@jdinh8124 jdinh8124 merged commit d2f2c68 into main Sep 6, 2024
16 checks passed
Copy link
Contributor

github-actions bot commented Sep 6, 2024

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

The release is available on GitHub release

Your semantic-release bot 📦🚀

@jdinh8124 jdinh8124 deleted the g2-divider branch September 16, 2024 22:31
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 type: FIX Submit bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants