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

Convert Mattermost markdown formatting / emphasis to IRC #547

Merged
merged 6 commits into from
Sep 29, 2023

Conversation

hloeung
Copy link
Collaborator

@hloeung hloeung commented Sep 17, 2023

Do the reverse of #546 for #258, convert Mattermost posts with markdown formatting / emphasis to IRC. Example:

Testing **Bold** Testing **_Bold+Italics_** Testing _Italics_ Testing `monospace` Testing

Becomes:

N8de65oriH
(FWIW, monospace support was added in irssi 1.4.4, I'm running 1.4.3 and why it's showing as Q)

@hloeung hloeung force-pushed the convert-formatting-mattermost branch 2 times, most recently from f9bce48 to c0884d2 Compare September 17, 2023 05:03
@hloeung hloeung force-pushed the convert-formatting-mattermost branch from c0884d2 to 010ab9e Compare September 17, 2023 05:06
Not all IRC clients support this yet, let's just drop it for now to
avoid having weird and funny characters appearing.
@hloeung hloeung force-pushed the convert-formatting-mattermost branch 4 times, most recently from 08da552 to 4906ae0 Compare September 18, 2023 14:00
@hloeung hloeung force-pushed the convert-formatting-mattermost branch from 4906ae0 to 98a3ead Compare September 18, 2023 14:01
@hloeung hloeung requested a review from 42wim September 20, 2023 05:40
@hloeung hloeung marked this pull request as ready for review September 27, 2023 08:16
@42wim
Copy link
Owner

42wim commented Sep 27, 2023

Looks good, I'm wondering if it should be an option for people who want to disable it

@hloeung hloeung force-pushed the convert-formatting-mattermost branch from b8b64b8 to 3bc2cf3 Compare September 27, 2023 10:45
@hloeung
Copy link
Collaborator Author

hloeung commented Sep 27, 2023

Sure, made an option to disable it. Thanks and much appreciated.

Copy link
Owner

@42wim 42wim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thx

@hloeung hloeung merged commit bbe3e54 into 42wim:master Sep 29, 2023
4 checks passed
@hloeung hloeung deleted the convert-formatting-mattermost branch September 29, 2023 20:28
@42wim 42wim added this to the 0.28.0 milestone Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants