Together-Java / TJ-Bot

TJ-Bot is a Discord Bot used on the Together Java server. It is maintained by the community, anyone can contribute.
https://togetherjava.org
GNU General Public License v3.0
100 stars 87 forks source link

Enhancing transfer-feature with media support and more #993

Open ankitsmt211 opened 10 months ago

ankitsmt211 commented 10 months ago

Is your feature request related to a problem? Please describe. Current implementation has a few problems,

Describe the solution you'd like

Describe alternatives you've considered

Additional context

JDA offers MessageHistory check it out.

Note: It might not always be a good idea to populate modal till the latest message from user, if texts are transferred a bit later it will just contain noise. Possibly pick up till there's messages from same user aka OP, as soon as the chain(of texts) continues it's safe to assume it will have relevant content.

Taz03 commented 5 months ago

image right now the modal shows the message in question to do edits (if needed), how the UI is going to look with multiple messages? Should we just remove the message section entirely?

ankitsmt211 commented 5 months ago

The goal is to move as much context as we can within the modal limit. So the question section should stay but we squeeze things in 2k char limit not sure if that's enough or if we should consider alternative options.

Although on the second note, we don't really edit questions for OP. Just the title and tag so maybe it's worth thinking about what you're suggesting.

ankitsmt211 commented 5 months ago

@Zabuzard any thoughts on this? any downsides for removing questions field?