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

Add clear chat memory command #48

Merged
merged 6 commits into from
Jan 25, 2025

Conversation

jaykayudo
Copy link
Contributor

Closes #27

Description

Add command for clearing chat memory

  • Clear chat history
  • Clear chat session

Copy link

vercel bot commented Jan 24, 2025

@jaykayudo is attempting to deploy a commit to the poulavbhowmick03's projects Team on Vercel.

A member of the Team first needs to authorize it.

@PoulavBhowmick03
Copy link
Contributor

@jaykayudo can you provide a working screenshot or video of the implementation locally?

Copy link
Contributor

@tosoham tosoham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jaykayudo just wanted to point don't change the bot.js. Do your changes in bot.ts it is the main file... the bot.js is just the result created after doing npx tsc in the directory.

@PoulavBhowmick03
Copy link
Contributor

@jaykayudo Can you please share the screenshot AND video recording of the changes?

@jaykayudo
Copy link
Contributor Author

Screencast.from.2025-01-24.20-41-14.webm

@PoulavBhowmick03 @tosoham
Please review. I believe i have completed the task.

@jaykayudo jaykayudo requested a review from tosoham January 24, 2025 19:49
Copy link
Contributor

@PoulavBhowmick03 PoulavBhowmick03 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
@tosoham WDYT?

Copy link
Contributor

@PoulavBhowmick03 PoulavBhowmick03 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tosoham
Copy link
Contributor

tosoham commented Jan 25, 2025

LGTM

@tosoham tosoham merged commit 573c770 into Shonen-Labs:main Jan 25, 2025
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Command to Clear Chat Memory
4 participants