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

fix: fix the bug "the prompt.id in conversationKey are undefined" #521

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

qwlong
Copy link
Contributor

@qwlong qwlong commented Mar 1, 2024

Now it's like:
--------conversationKey start---------
provider.id:undefined
--------conversationKey end---------

In fact, it should be
--------conversationKey start---------
provider.id:prompt.id
--------conversationKey end---------

Now it's like:
--------conversationKey start---------
provider.id:undefined
--------conversationKey end---------

In fact, it should be
--------conversationKey start---------
provider.id:prompt.id
--------conversationKey end---------
@qwlong qwlong changed the title Fix the bug: the prompt.id in conversationKey are undefined fix: fix the bug "the prompt.id in conversationKey are undefined" Mar 1, 2024
@typpo
Copy link
Collaborator

typpo commented Mar 2, 2024

Thanks @qwlong! Looks like there are some test failures we need to look at

@qwlong
Copy link
Contributor Author

qwlong commented Mar 14, 2024

@typpo
when would you merge this pull request?

@mldangelo mldangelo force-pushed the main branch 2 times, most recently from 99b5c63 to 790ec30 Compare June 8, 2024 19:41
@typpo typpo force-pushed the main branch 11 times, most recently from b2323de to 3ad344f Compare June 18, 2024 18:36
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.

None yet

2 participants