Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Strip topic name when formatting RPL_HELPSTART/RPL_HELPTXT/RPL_ENDOFHELP/ERR_HELPNOTFOUND - #1772

Closed
progval wants to merge 2 commits into
weechat:masterfrom
progval:help
Closed

Strip topic name when formatting RPL_HELPSTART/RPL_HELPTXT/RPL_ENDOFHELP/ERR_HELPNOTFOUND#1772
progval wants to merge 2 commits into
weechat:masterfrom
progval:help

Conversation

@progval

@progval progval commented Apr 1, 2022

Copy link
Copy Markdown
Contributor

For example, when using /quote help mode on Ergo:

Before:

screenshot-2022-04-01_17-36-41

After:

screenshot-2022-04-01_17-53-36

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #1772 (f3944d8) into master (9b1184e) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1772      +/-   ##
==========================================
- Coverage   37.99%   37.99%   -0.01%     
==========================================
  Files         210      210              
  Lines       85271    85280       +9     
==========================================
  Hits        32401    32401              
- Misses      52870    52879       +9     
Impacted Files Coverage Δ
src/plugins/irc/irc-protocol.c 87.52% <0.00%> (-0.35%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9b1184e...f3944d8. Read the comment docs.

@progval progval changed the title Strip topic name when formatting RPL_HELPSTART/RPL_HELPTXT/RPL_ENDOFHELP Strip topic name when formatting RPL_HELPSTART/RPL_HELPTXT/RPL_ENDOFHELP/ERR_HELPNOTFOUND Apr 6, 2022
@flashcode

Copy link
Copy Markdown
Member

Hi,
Thanks for your PR.
Could you please tell me on which server and the command to get this output?

@flashcode flashcode self-assigned this Jul 2, 2022
@flashcode flashcode added the bug Unexpected problem or unintended behavior label Jul 2, 2022
@progval

progval commented Jul 2, 2022

Copy link
Copy Markdown
Contributor Author

IIRC that was on irc.ergo.chat/6697 when typing /quote help mode

@flashcode flashcode added feature New feature request and removed bug Unexpected problem or unintended behavior labels Jul 2, 2022
@flashcode flashcode added this to the 3.6 milestone Jul 2, 2022
flashcode pushed a commit that referenced this pull request Jul 2, 2022
flashcode pushed a commit that referenced this pull request Jul 2, 2022
flashcode added a commit that referenced this pull request Jul 2, 2022
flashcode added a commit that referenced this pull request Jul 2, 2022
@flashcode

Copy link
Copy Markdown
Member

Thanks, merged.

I added unit tests on these messages 524/704/705/706 (and found a bug with hook_print callback not called on empty messages, fixed as well).

@flashcode flashcode closed this Jul 2, 2022
@progval
progval deleted the help branch July 2, 2022 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants