-
Notifications
You must be signed in to change notification settings - Fork 397
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
Fixed multiple operators in Basic Calculator #906
Fixed multiple operators in Basic Calculator #906
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @tanu-agarwal0101, Welcome to the project CalcDiverse! 🎊
Thanks for your contribution! Your effort makes this project better. Keep it up! 🙌
Please wait for the PR to be reviewed. Happy Coding!! ✨
✅ Deploy Preview for calcdiverse ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
The issue is solved from the UI part, but when we enter input from keyboard, the issue is not fixed. Pls see to this @tanu-agarwal0101 Make the new commits in this same PR, don't create a new PR!! |
Ok sir |
@Rakesh9100 please check now |
Congrats on getting your PR merged to the repo under GSSOC 2k24 🥳🎉 @tanu-agarwal0101 |
Fixes Issue🛠️
Closes #685
Description👨💻
Multiple operators issue fixed.
Type of change📄
How this has been tested✅
Tested locally.
Checklist✅
Screenshots/GIF📷
Before
Now