Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Replies do not mention when toggled to.

See original GitHub issue

Make sure you familiarize yourself with our contributing guidelines.

Summary

With #704 merged, inline replies were added, as well as the DiscordMessage builder, however, using the builder to add replies do not set the requisite “replied_user” to true, which toggles the mention.

Details

This has been tested on Windows 10, and Ubuntu 20.04 LTS, both using .NET 5, nightly build #00777

Using CommandContext#RespondAsync(DiscordMessageBuilder) override does not properly mention, but DiscordEmbedBuilder#SendAsync(DiscordChannel) does.

Steps to reproduce

Create message builder in a command
Add content or embed so there’s a valid message
Add a reply of your choosing, and set reply to true
Send via the command context
Send via the builder
Observe.

Issue Analytics

State:
Created 3 years ago
Comments:6 (6 by maintainers)

Top GitHub Comments

1reaction

jeffreyladdcommented, Jan 14, 2021

Okay i found the cause, this was a bug way before the message builder was merged into the inline replies branch. Ill create a new new PR to fix this.

0reactions

JulianusIVcommented, Jan 14, 2021

Addition to this reply https://github.com/DSharpPlus/DSharpPlus/issues/731#issuecomment-760196784 no need for breaking as on PC there will be a @ infront of the name at the reply if it pinged, which is missing when it did not ping