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

Reject toPrompt Promise when 'exit' is emitted. #369

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zArubaru
Copy link

@zArubaru zArubaru commented Sep 9, 2022

Resolves #362

This fixes the 'autocomplete' prompt incorrectly firing onSubmit when it is exitted by pressing ESC. Now correctly fires onCancel.

This fixes the 'autocomplete' prompt incorrectly firing onSubmit when it
is exitted by pressing ESC. Now correctly fires onCancel.
@frenchbread
Copy link

Any ideas when this PR will be merged?

@crazyxhz
Copy link

crazyxhz commented Jun 4, 2023

is this project not maintained? such a big bug persist so long without merged

@TMaize
Copy link

TMaize commented Sep 13, 2023

Here is a fork to resolve it,and publish to npm

TakshPSingh added a commit to withgraphite/prompts that referenced this pull request Feb 5, 2024
this is an open issue in the library: terkelg#362

here's a fix for this that someone made a while ago (which has not been merged for some reason): terkelg#369

This PR makes the same fix in our internal fork
TakshPSingh added a commit to withgraphite/prompts that referenced this pull request Feb 5, 2024
this is an open issue in the library: terkelg#362

here's a fix for this that someone made a while ago (which has not been merged for some reason): terkelg#369

This PR makes the same fix in our internal fork
TakshPSingh added a commit to withgraphite/prompts that referenced this pull request Feb 5, 2024
this is an open issue in the library: terkelg#362

here's a fix for this that someone made a while ago (which has not been merged for some reason): terkelg#369

This PR makes the same fix to our fork
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.

Autocomplete submits on ESC.
4 participants