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.

Sign up for GitHub

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

Jump to bottom

streamalert list is not updated for new Discord's markdown #6273

Closed
madebylydia opened this issue Dec 2, 2023 · 1 comment · Fixed by #6292
Closed

streamalert list is not updated for new Discord's markdown #6273

madebylydia opened this issue Dec 2, 2023 · 1 comment · Fixed by #6292
Assignees
@palmtree5 @Kreusada
Labels
Category: Cogs - Streams This is related to the Streams cog. Status: In Progress There's a PR open to deal with this, or an org member is working on this internally Type: Enhancement Something meant to enhance existing Red features.

Comments

@madebylydia
Copy link
Contributor

What component of Red (cog, command, API) would you like to see improvements on?

Streams

Describe the enhancement you're suggesting.

Discord has updated Markdown to support header & lists, whatever else they've added too...

However, this broke the streamalert list command in the process, because the output of the command return a list that would not work well with Discord's new Markdown functionalities.

It would look like that:
image

But now look like this
image

Feel free to suggest a new output syntax to respect.
Either rewrite the command output, or simply escape the markdown for the "-" character.

Anything else?

No response
@madebylydia madebylydia added the Type: Enhancement Something meant to enhance existing Red features. label Dec 2, 2023
@github-actions github-actions bot added the Status: Needs Triage This has not been labeled or discussed for handling yet. label Dec 2, 2023
@palmtree5 palmtree5 self-assigned this Dec 2, 2023
@palmtree5 palmtree5 added Status: Accepted We want this and removed Status: Needs Triage This has not been labeled or discussed for handling yet. labels Dec 2, 2023
@palmtree5 palmtree5 added this to the 3.5.6 milestone Dec 2, 2023
@palmtree5
Copy link
Member

I'll attempt to work on this this weekend. It seems like it should be an easy fix

@Kreusada Kreusada self-assigned this Feb 16, 2024
@Kreusada Kreusada mentioned this issue Feb 16, 2024
Merged
@Kreusada Kreusada added Status: In Progress There's a PR open to deal with this, or an org member is working on this internally Category: Cogs - Streams This is related to the Streams cog. and removed Status: Accepted We want this labels Feb 16, 2024
@Jackenmen Jackenmen removed this from the 3.5.6 milestone Mar 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@palmtree5 palmtree5

@Kreusada Kreusada

Labels
Category: Cogs - Streams This is related to the Streams cog. Status: In Progress There's a PR open to deal with this, or an org member is working on this internally Type: Enhancement Something meant to enhance existing Red features.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix markdown formatting in [p]streamalert list Kreusada/Red-DiscordBot
4 participants
@palmtree5 @Jackenmen @madebylydia @Kreusada