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

Adds logic to prevent multiple submissions when enter is pressed #2208

Closed
wants to merge 7 commits into from

Conversation

navaneethsdk
Copy link
Contributor

@navaneethsdk navaneethsdk commented May 29, 2024

Description
Adds logic to prevent multiple submissions when enter is pressed
Video: https://navaneeth-d.neetorecord.com/watch/91e0358f-dbaf-468a-a615-09bf4cfc90dc

Checklist

- [ ] I have made corresponding changes to the documentation.
- [ ] I have updated the types definition of modified exports.
- [ ] I have verified the functionality in some of the neeto web-apps.
- [ ] I have added tests that prove my fix is effective or that my feature works.
- [ ] I have added proper data-cy and data-testid attributes.

  • I have added the necessary label (patch/minor/major - If package publish
    is required).

Reviewers

@neetodeploy neetodeploy bot temporarily deployed to neeto-ui-kpyu-pr-2208 May 29, 2024 14:15 Inactive
@navaneethsdk
Copy link
Contributor Author

@josephmathew900 _a Please review. Please have a look at the video description for more context.

Copy link
Contributor

neetogit-bot bot commented May 29, 2024

@navaneethsdk CI is red.

* Replaces tooltip with Popover in Label

* Modifies test

* Replaced text content with node

* Adds support for both tooltip and popover

* Incorporates specific popover props

* Review changes

* Fixes minor bug
@josephmathew900
Copy link
Contributor

@navaneethsdk _a Please fix the CI and assign the PR back to me.

nitinprojects and others added 4 commits June 1, 2024 23:20
* Fixed alignment of thumbs up toastr

* Added back neeto-ui-toastr class

* Removed unnecessary !important usage from CSS
@neetodeploy neetodeploy bot temporarily deployed to neeto-ui-kpyu-pr-2208 June 3, 2024 11:38 Inactive
@neetodeploy neetodeploy bot requested a deployment to neeto-ui-kpyu-pr-2208 June 3, 2024 11:40 In progress
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.

Multiple API requests on pressing Enter key multiple times
4 participants