-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
R4R: Make tags and responses legible #3451
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #3451 +/- ##
========================================
Coverage 52.69% 52.69%
========================================
Files 142 142
Lines 10364 10364
========================================
Hits 5461 5461
Misses 4574 4574
Partials 329 329 |
Ok, so I have the |
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.
@jackzampolin this is great and I love the approach taken here. I think we can cleanup some of the logic even further (see my comment).
98c2349
to
31358e9
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.
Looks great @jackzampolin -- tested locally. I left a few comments.
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.
TestedACK (on a tx send
)
@jackzampolin this needs a pending log entry and merge conflict resolution. |
fc85e98
to
c49d580
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.
Just a small change - looks good otherwise
Will merge when tests pass cc @alessio |
Replaces: #3277
docs/
)PENDING.md
with issue #Files changed
in the github PR explorerFor Admin Use: