Skip to content

fix: close notification with Enter key#301

Open
MehmetYararVX wants to merge 2 commits into
react-component:4.xfrom
MehmetYararVX:4.x
Open

fix: close notification with Enter key#301
MehmetYararVX wants to merge 2 commits into
react-component:4.xfrom
MehmetYararVX:4.x

Conversation

@MehmetYararVX

Copy link
Copy Markdown
Contributor

fixes #299

@vercel

vercel Bot commented Mar 1, 2023

Copy link
Copy Markdown

@MehmetYararVX is attempting to deploy a commit to the React Component Team on Vercel.

A member of the Team first needs to authorize it.

Comment thread tests/index.test.js
duration: null,
});

setTimeout(() => {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Use waitFakeTimers.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@yoyo837, all the tests use setTimeout in this test suite, and this test is no different. Not sure this PR is the right place to refactor.

@zombieJ

zombieJ commented Mar 1, 2023

Copy link
Copy Markdown
Member

strange, why CI not trigger...

@mellis481

Copy link
Copy Markdown

@zombieJ Are you able to determine why the workflow wasn't kicked off successfully?

@zombieJ

zombieJ commented Mar 3, 2023

Copy link
Copy Markdown
Member

@zombieJ Are you able to determine why the workflow wasn't kicked off successfully?

Could you push some commit to force trigger CI for this?

@zombieJ

zombieJ commented Mar 6, 2023

Copy link
Copy Markdown
Member

Hmm... got the reason. Could you help merge into master instead?

@MehmetYararVX

Copy link
Copy Markdown
Contributor Author

Hmm... got the reason. Could you help merge into master instead?

@zombieJ #302

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.

4 participants