-
Notifications
You must be signed in to change notification settings - Fork 78
feat(FR-1495): implement progress bar with bai notifiacation #4264
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
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has required the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
|---|---|---|---|
| 🔴 | Statements | 52.42% | 130/248 |
| 🔴 | Branches | 32.26% | 80/248 |
| 🔴 | Functions | 35.09% | 20/57 |
| 🔴 | Lines | 54.84% | 119/217 |
Test suite run success
55 tests passing in 3 suites.
Report generated by 🧪jest coverage report action from 56918ce
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
|---|---|---|---|
| 🔴 | Statements | 4.49% (-0% 🔻) |
501/11167 |
| 🔴 | Branches | 3.64% | 285/7829 |
| 🔴 | Functions | 2.59% (-0% 🔻) |
90/3481 |
| 🔴 | Lines | 4.46% (-0% 🔻) |
487/10917 |
Test suite run success
114 tests passing in 13 suites.
Report generated by 🧪jest coverage report action from 56918ce
cb9ef8f to
435c1fd
Compare
6d2d9fc to
04c97d6
Compare
435c1fd to
be8d14e
Compare
04c97d6 to
46589d0
Compare
46589d0 to
1dffa87
Compare
c7a5adc to
4bbd42b
Compare
1dffa87 to
01fcc67
Compare
4bbd42b to
7962b07
Compare
01fcc67 to
b9232a0
Compare
7962b07 to
cc94717
Compare
b9232a0 to
58c1c32
Compare
58c1c32 to
5aedb3a
Compare
cc94717 to
482502f
Compare
482502f to
30b7e13
Compare
5aedb3a to
25530b2
Compare
25530b2 to
1aeaa1b
Compare
30b7e13 to
bf46406
Compare
bf46406 to
d977e7c
Compare
1aeaa1b to
bd95558
Compare
d977e7c to
f28bdee
Compare
bd95558 to
dcea61c
Compare
f28bdee to
711915a
Compare
dcea61c to
4d203b0
Compare
711915a to
d34ace3
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.
LGTM
Merge activity
|
resolves #4307 (FR-1495) <!-- replace NNN, MMM with the GitHub issue number and the corresponding Jira issue number. --> <!-- Please precisely, concisely, and concretely describe what this PR changes, the rationale behind codes, and how it affects the users and other developers. --> **Checklist:** (if applicable) - [ ] Documentation - [ ] Minium required manager version - [ ] Specific setting for review (eg., KB link, endpoint or how to setup) - [ ] Minimum requirements to check during review - [ ] Test case(s) to demonstrate the difference of before/after
27c9bf6 to
3aca6b9
Compare
d34ace3 to
56918ce
Compare

resolves #4307 (FR-1495)
Checklist: (if applicable)