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

[Bug] : AI reply modal don't have last users message #40

Open
not-meet opened this issue Feb 24, 2025 · 3 comments · May be fixed by #44
Open

[Bug] : AI reply modal don't have last users message #40

not-meet opened this issue Feb 24, 2025 · 3 comments · May be fixed by #44
Labels
bug Something isn't working

Comments

@not-meet
Copy link
Contributor

What happened?

The bug relies in the /quick ai modal as when you open the modal, It has the last room message in the modal to create Ai based reply rather then Last message of the other user

Steps to reproduce

  1. use /quick ai command
  2. notice when the modal opens it has last room message in the modal to generate AI reply for

Relevant ScreenShots

Screencast.from.2025-02-25.04-47-09.webm

Here is a small demo i tried and tested this in more than two channels and faced the same issue

Version

v7.2.0

What browsers are you seeing the problem on?

Chrome
@not-meet not-meet added the bug Something isn't working label Feb 24, 2025
@not-meet
Copy link
Contributor Author

hey @VipinDevelops i believe it will be really great if the modal have the last reply from the other user to generate AI made reply...
From my pov
if we add a condition that only the last reply which have different user name is considered to generate reply for would be a great addition what do you say?.

@VipinDevelops
Copy link
Collaborator

yeah, @not-meet Handle all the edge case and also show user some output that why they are not able to get a out put

@not-meet
Copy link
Contributor Author

i am planning to send user a notification if there is no reply other user reply found so in that moment rather than opening ai reply modal the notification will be send to the user No reply found or something

Ill make a poc and will add a pr as soon as possible !

@not-meet not-meet linked a pull request Feb 26, 2025 that will close this issue
Open
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[FIX] : Filtered out self messages from AI reply generation not-meet/Apps.QuickReplies
2 participants
@VipinDevelops @not-meet