-
Notifications
You must be signed in to change notification settings - Fork 44
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
added dynamic links to buttons #3180
Conversation
🚀 Deployed on https://deploy-preview-3180--glific-frontend.netlify.app |
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.
please check on test failures.
Looks like now the tests are failing as other PR is merged. Can you please check. |
Glific Run #5501
Run Properties:
|
Project |
Glific
|
Branch Review |
template/dynamic-url
|
Run status |
Passed #5501
|
Run duration | 25m 36s |
Commit |
a82dc582b2 ℹ️: Merge 8a3547b1fe994bf21166c1bc6befbfb2103291e1 into 3e82e09a95b20041c02baf091f1f...
|
Committer | Akansha Sakhre |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
0
|
Pending |
0
|
Skipped |
0
|
Passing |
172
|
View all changes introduced in this branch ↗︎ |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #3180 +/- ##
==========================================
- Coverage 81.85% 81.82% -0.03%
==========================================
Files 327 327
Lines 10526 10539 +13
Branches 2174 2177 +3
==========================================
+ Hits 8616 8624 +8
- Misses 1300 1303 +3
- Partials 610 612 +2 ☔ View full report in Codecov by Sentry. |
target issue is #3157