Skip to content

[$15] [Web- Chrome] While filtering with price with o to 2,0000 app displaying message as invalid format #96

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

Closed
LieutenantRoger opened this issue Nov 12, 2021 · 28 comments

Comments

@LieutenantRoger
Copy link
Collaborator

Summary :
[Web- Chrome] While filtering with price with o to 2,0000 app displaying message as invalid format

Steps to reproduce:

  1. Open the https://platform.topcoder.com/earn/find/challenges
  2. In price as o to 2,0000

Actual:
[Web- Chrome] While filtering with price with o to 2,0000 app displaying message as invalid format

Expected:
App should accept, app should auto correct and display the correct format of price

Environment:

_ Device(s): Windows
_ Resolution: 1920_1080
_ Operating System: Windows 10
_ Browser(s): Chrome | Version 95.0.4638.54 (| Firefox 93.0 (64-bit)| Edge 95.0.1020.30

image
bandicam_2021-10-29_08-10-50-317

@LieutenantRoger LieutenantRoger changed the title [Web- Chrome] While filtering with price with o to 2,0000 app displaying message as invalid format [$15] [Web- Chrome] While filtering with price with o to 2,0000 app displaying message as invalid format Nov 16, 2021
@LieutenantRoger
Copy link
Collaborator Author

Challenge https://www.topcoder.com/challenges/f14575a4-d111-4e8b-8964-65b0422151e3 has been created for this ticket.

This is an automated message for lieutenantroger via Topcoder X

@shivam-51
Copy link
Contributor

@bug-bash-helper assign

@bug-bash-helper
Copy link

@shivam-51 🛑 you have some issues assigned which are not completed yet, see #132.

As per our Bug Bash rules you may work only at one issue at a time.
Please, complete other issues first or unassign yourself before picking up a new issue.

1 similar comment
@bug-bash-hunt-helper
Copy link

@shivam-51 🛑 you have some issues assigned which are not completed yet, see #132.

As per our Bug Bash rules you may work only at one issue at a time.
Please, complete other issues first or unassign yourself before picking up a new issue.

@shivam-51
Copy link
Contributor

@bug-bash-helper assign

@bug-bash-helper
Copy link

@shivam-51 ✅ you are now assigned to this issue and have 12 hours to complete it.

As soon as you are done, please, make a comment like below, including the link to the pull request:

@bug-bash-helper <link to PR> is ready for review

@bug-bash-hunt-helper
Copy link

@shivam-51 ✅ you are now assigned to this issue and have 12 hours to complete it.

As soon as you are done, please, make a comment like below, including the link to the pull request:

@bug-bash-helper <link to PR> is ready for review

@LieutenantRoger
Copy link
Collaborator Author

Challenge https://www.topcoder.com/challenges/f14575a4-d111-4e8b-8964-65b0422151e3 has been assigned to shivam51.

This is an automated message for lieutenantroger via Topcoder X

@shivam-51
Copy link
Contributor

@bug-bash-helper #148 is ready for review.

@bug-bash-helper
Copy link

@shivam-51 ✅ this issue is marked as Ready for Review.

Now you may pick up another issue which is open for pickup if you like to.

1 similar comment
@bug-bash-hunt-helper
Copy link

@shivam-51 ✅ this issue is marked as Ready for Review.

Now you may pick up another issue which is open for pickup if you like to.

@LieutenantRoger
Copy link
Collaborator Author

@shivam-51

It looks you are changing the component's prop directly in the onChange function, which is not correct. Component should not update its own property directly

@shivam-51
Copy link
Contributor

Noted. Working on it.

@shivam-51
Copy link
Contributor

@bug-bash-helper #148 is ready for review

@LieutenantRoger
Copy link
Collaborator Author

Challenge https://www.topcoder.com/challenges/f14575a4-d111-4e8b-8964-65b0422151e3 has been assigned to shivam51.

This is an automated message for lieutenantroger via Topcoder X

@shivam-51
Copy link
Contributor

@bug-bash-helper unassign

@bug-bash-hunt-helper
Copy link

@shivam-51 ✅ you have been unassigned from this issue.

Now you may pick up another issue which is open for pickup if you like to.

@LieutenantRoger
Copy link
Collaborator Author

Challenge https://www.topcoder.com/challenges/f14575a4-d111-4e8b-8964-65b0422151e3 shivam51 has been unassigned.

This is an automated message for lieutenantroger via Topcoder X

@LieutenantRoger
Copy link
Collaborator Author

Challenge https://www.topcoder.com/challenges/f14575a4-d111-4e8b-8964-65b0422151e3 has been assigned to shivam51.

This is an automated message for lieutenantroger via Topcoder X

@codejamtc
Copy link
Collaborator

Verified in Prod 🆗

bandicam.2021-11-23.11-10-36-476.mp4

@LieutenantRoger
Copy link
Collaborator Author

process payment manually

LieutenantRoger added a commit that referenced this issue Dec 4, 2021
* issue #145

* fix: issue #110

* Fixed #96

* Reset Pagination to 1 When Choosing Filter

* linter

* fix: issue #126

* fix: issue #106

* issue #143

* Prevent Trigger Change On Date Picker Input

* fix: issue #85

* Fixes #95

* fix: issue #76

* issue #97

* ci:deploying

* Add Close Button in Calendar

* issue #132

* issue 132: focus selection range when having value

* issue 132: fixed duration of a week having 8 days

* Clear Challenge Filter On Menu Click

* Reset Pagination to 1

* linter

* Implement Not Found Error

* Use debounce function for sort by

* issue #75

* Minor update completed

* Updated totalPrizesTo

* issue 75: refixed

* fix: issue #76

* fix: issue #85

* ci:deploying

* Fixes #95

* issue #75

* issue 75: refixed

* Use debounce function for sort by

* issue #97

* Reset Pagination to 1 When Choosing Filter

* linter

* issue #145

* Implement Not Found Error

* Prevent Trigger Change On Date Picker Input

* issue #132

* issue 132: focus selection range when having value

* issue 132: fixed duration of a week having 8 days

* =resolve merge conflict

* Replace Track QA

* Reset Pagination

* Minor updates done for fixing #96

* fix: lint

* fix: refine

* restore ci

* deploy bug bash

* issue 91 fix (#169)

* issue 91 fix

* feedback fix

* adjustment

* restore ci

* issue #115 (#183)

* ci: deploying

* fixed some issues (#184)

* fixed issues:
- detailed page navigation should not refresh the page
- Joi validation of url query should return default values if not matched
- missing QA track icon

* fixed track -> [track]

* ci: revert

* issue #132: fixed focused date jump (#191)

* fix lint

* ci:deploying

* Fix Issue 182 (#199)

* Add loading indicator

* update margin

Co-authored-by: mfikria <[email protected]>

* DO not reset challenges (#200)

* <fix> issue 102: submit without registration (#197)

* fix: issue 133 (#203)

* <fix> issue_133

* <fix> issue_133 - code updated

* revert ci

* fix lint

* fix:254

* issue #258 (#279)

* ci: deploying challengeapp

* fixed responsive (#280)

* fixed responsive

* fixed responsive for ipad (768px)

* fixed title font size

* fixed clicking mobile menu

* wrap code (#286)

* Issue#228 (#270)

* fixes #228

* Update index.jsx for #228

* Update index.js for #228

* fix lint

* clear btn

* handle

* fixes #226 (#290)

* fix price

* handle width

* reset ci

Co-authored-by: Nguyen Viet <[email protected]>
Co-authored-by: yoution <[email protected]>
Co-authored-by: Shivam Kumar Singh <[email protected]>
Co-authored-by: M Fikri A <[email protected]>
Co-authored-by: Nguyen Viet <[email protected]>
Co-authored-by: Gaurav Seta <[email protected]>
Co-authored-by: M Fikri A <[email protected]>
Co-authored-by: mfikria <[email protected]>
Co-authored-by: linmiao <[email protected]>
LieutenantRoger added a commit that referenced this issue Dec 7, 2021
* [Bug Bash] Fix 1  (#173)

* issue #145

* fix: issue #110

* Fixed #96

* Reset Pagination to 1 When Choosing Filter

* linter

* fix: issue #126

* fix: issue #106

* issue #143

* Prevent Trigger Change On Date Picker Input

* fix: issue #85

* Fixes #95

* fix: issue #76

* issue #97

* ci:deploying

* Add Close Button in Calendar

* issue #132

* issue 132: focus selection range when having value

* issue 132: fixed duration of a week having 8 days

* Clear Challenge Filter On Menu Click

* Reset Pagination to 1

* linter

* Implement Not Found Error

* Use debounce function for sort by

* issue #75

* Minor update completed

* Updated totalPrizesTo

* issue 75: refixed

* fix: issue #76

* fix: issue #85

* ci:deploying

* Fixes #95

* issue #75

* issue 75: refixed

* Use debounce function for sort by

* issue #97

* Reset Pagination to 1 When Choosing Filter

* linter

* issue #145

* Implement Not Found Error

* Prevent Trigger Change On Date Picker Input

* issue #132

* issue 132: focus selection range when having value

* issue 132: fixed duration of a week having 8 days

* =resolve merge conflict

* Replace Track QA

* Reset Pagination

* Minor updates done for fixing #96

* fix: lint

* fix: refine

* restore ci

Co-authored-by: Nguyen Viet <[email protected]>
Co-authored-by: yoution <[email protected]>
Co-authored-by: Shivam Kumar Singh <[email protected]>
Co-authored-by: M Fikri A <[email protected]>
Co-authored-by: Nguyen Viet <[email protected]>

* Challenges bug bash - three addition tickets (#178)

Three addition bug tickets

* [Bug Bash] Challenges bug bash - One more ticket (#185)

* [Additional Fix] Challenges bug bash (#192)

Additional fix for issue_132

* Commented out Grid View Button (#195)

Co-authored-by: Pauline <[email protected]>

* Revert "Commented out Grid View Button (#195)" (#196)

This reverts commit c0efb7e.

* [Bug Bash] Challenges bug bash - 2 (#267)

* [Bug Bash] Fixign lint

* fixes #254 (#275)

* Revert "fixes #254 (#275)" (#282)

This reverts commit 76bb2f4.

* [Bug Bash] Challenges bug bash - Round 3 (#287)

* issue #145

* fix: issue #110

* Fixed #96

* Reset Pagination to 1 When Choosing Filter

* linter

* fix: issue #126

* fix: issue #106

* issue #143

* Prevent Trigger Change On Date Picker Input

* fix: issue #85

* Fixes #95

* fix: issue #76

* issue #97

* ci:deploying

* Add Close Button in Calendar

* issue #132

* issue 132: focus selection range when having value

* issue 132: fixed duration of a week having 8 days

* Clear Challenge Filter On Menu Click

* Reset Pagination to 1

* linter

* Implement Not Found Error

* Use debounce function for sort by

* issue #75

* Minor update completed

* Updated totalPrizesTo

* issue 75: refixed

* fix: issue #76

* fix: issue #85

* ci:deploying

* Fixes #95

* issue #75

* issue 75: refixed

* Use debounce function for sort by

* issue #97

* Reset Pagination to 1 When Choosing Filter

* linter

* issue #145

* Implement Not Found Error

* Prevent Trigger Change On Date Picker Input

* issue #132

* issue 132: focus selection range when having value

* issue 132: fixed duration of a week having 8 days

* =resolve merge conflict

* Replace Track QA

* Reset Pagination

* Minor updates done for fixing #96

* fix: lint

* fix: refine

* restore ci

* deploy bug bash

* issue 91 fix (#169)

* issue 91 fix

* feedback fix

* adjustment

* restore ci

* issue #115 (#183)

* ci: deploying

* fixed some issues (#184)

* fixed issues:
- detailed page navigation should not refresh the page
- Joi validation of url query should return default values if not matched
- missing QA track icon

* fixed track -> [track]

* ci: revert

* issue #132: fixed focused date jump (#191)

* fix lint

* ci:deploying

* Fix Issue 182 (#199)

* Add loading indicator

* update margin

Co-authored-by: mfikria <[email protected]>

* DO not reset challenges (#200)

* <fix> issue 102: submit without registration (#197)

* fix: issue 133 (#203)

* <fix> issue_133

* <fix> issue_133 - code updated

* revert ci

* fix lint

* fix:254

* issue #258 (#279)

* ci: deploying challengeapp

* fixed responsive (#280)

* fixed responsive

* fixed responsive for ipad (768px)

* fixed title font size

* fixed clicking mobile menu

* wrap code (#286)

* Issue#228 (#270)

* fixes #228

* Update index.jsx for #228

* Update index.js for #228

* fix lint

* clear btn

* handle

* fixes #226 (#290)

* fix price

* handle width

* reset ci

Co-authored-by: Nguyen Viet <[email protected]>
Co-authored-by: yoution <[email protected]>
Co-authored-by: Shivam Kumar Singh <[email protected]>
Co-authored-by: M Fikri A <[email protected]>
Co-authored-by: Nguyen Viet <[email protected]>
Co-authored-by: Gaurav Seta <[email protected]>
Co-authored-by: M Fikri A <[email protected]>
Co-authored-by: mfikria <[email protected]>
Co-authored-by: linmiao <[email protected]>

* fix merge issue

* [Bug Bash] Challenges bug bash - Additional Two tickets (#318)

Co-authored-by: Nguyen Viet <[email protected]>
Co-authored-by: yoution <[email protected]>
Co-authored-by: Shivam Kumar Singh <[email protected]>
Co-authored-by: M Fikri A <[email protected]>
Co-authored-by: Nguyen Viet <[email protected]>
Co-authored-by: pauline <[email protected]>
Co-authored-by: Pauline <[email protected]>
Co-authored-by: Gaurav Seta <[email protected]>
Co-authored-by: M Fikri A <[email protected]>
Co-authored-by: mfikria <[email protected]>
Co-authored-by: linmiao <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants