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

Enhancing transfer-feature with media support and more #993

Open
ankitsmt211 opened this issue Dec 18, 2023 · 3 comments
Open

Enhancing transfer-feature with media support and more #993

ankitsmt211 opened this issue Dec 18, 2023 · 3 comments
Labels
enhancement New feature or request priority: low

Comments

@ankitsmt211
Copy link
Member

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

  • No media support
  • Sometimes leaves a couple of messages from user when there's multiple single messages for instance

Describe the solution you'd like

  • It should add media from user to thread as well
  • Let's say user sends 10 small messages instead of a paragraph of text, all those texts from the start to latest should populate modal menu and post successful transfer should be deleted from chat.

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.

@ankitsmt211 ankitsmt211 added enhancement New feature or request priority: low labels Dec 18, 2023
@Taz03
Copy link
Member

Taz03 commented May 15, 2024

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
Copy link
Member Author

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
Copy link
Member Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority: low
Projects
Status: Ready
Development

No branches or pull requests

2 participants