-
Notifications
You must be signed in to change notification settings - Fork 831
CHR-719/scrollable prompt history #1238
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
base: feat/attach-image-on-path-auto-complete-selection
Are you sure you want to change the base?
CHR-719/scrollable prompt history #1238
Conversation
Signed-off-by: Carlos Alexandro Becker <[email protected]>
Signed-off-by: Carlos Alexandro Becker <[email protected]>
Signed-off-by: Carlos Alexandro Becker <[email protected]>
Signed-off-by: Carlos Alexandro Becker <[email protected]>
Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…7.0.0 in the all group (#1226) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
… CHR-719/scrollable-prompt-history
| if len(m.historyCache) > 0 { | ||
| return m.historyCache, nil | ||
| } | ||
| allMessages, err := m.app.Messages.List(ctx, m.session.ID) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The history here should be build across sessions it does not have to be just the current session, imagine you create a new session but want the last sessions message but just make it a bit different.
To enable this you need to create a new SQL query and add a new method to the service that lists all user messages with a specific order (newer messages show up first)
No description provided.