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

feat: added runner delay #2218

Merged
merged 4 commits into from
Aug 8, 2024
Merged

Conversation

therealrinku
Copy link
Contributor

Description

Added delay time for runner
resolves: #1385

Contribution Checklist:

  • The pull request only addresses one issue or adds one feature.
  • The pull request does not introduce any breaking changes
  • I have added screenshots or gifs to help explain the change if applicable.
  • I have read the contribution guidelines.
  • Create an issue and link to the pull request.
Screencast.from.2024-05-02.14-25-59.webm

@helloanoop
Copy link
Contributor

Thank you @therealrinku !

@lohxt1 Can you review this?

</span>
) : (
<span className="danger text-xs">(request failed)</span>
Copy link
Collaborator

Choose a reason for hiding this comment

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

could you also add the onClick handler for the 'request failed' span

onClick={() => setSelectedItem(item)}

<div className="mt-6">
<label>Delay (in ms)</label>
<input
type="text"
Copy link
Collaborator

Choose a reason for hiding this comment

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

would you mind changing this to the type 'number'

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sure, done!

@therealrinku therealrinku requested a review from lohxt1 June 18, 2024 05:55
@jroucou
Copy link

jroucou commented Jul 17, 2024

hello,
is it possible to provide a parameter for bruno-cli as well ? could be a nice addition.

@helloanoop helloanoop merged commit de226d2 into usebruno:main Aug 8, 2024
3 checks passed
@helloanoop
Copy link
Contributor

Merged! Thanks @therealrinku !

@therealrinku therealrinku deleted the feat/runner-delay branch August 8, 2024 13:06
Its-treason added a commit to Its-treason/bruno that referenced this pull request Aug 10, 2024
@bstupid
Copy link

bstupid commented Sep 12, 2024

This menu option is not available when executing runner from the collection / run vs Runner top of the screen.

image

@feanor777
Copy link

hello, is it possible to provide a parameter for bruno-cli as well ? could be a nice addition.

also looking forward for this feature!

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.

[Feature] add delay in run collection
6 participants