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

Job Status tab bugs bugs #4867

Closed
oritwoen opened this issue Jul 25, 2019 · 5 comments
Closed

Job Status tab bugs bugs #4867

oritwoen opened this issue Jul 25, 2019 · 5 comments
Labels
bug This is something that isn't working as intended.

Comments

@oritwoen
Copy link
Contributor

Describe the bug

A little honestly saying these bugs made me laugh, act quite specifically. It is about bugs related to the "Job Status" tab and its updating. They have quite an interesting logic of action.

  • Bug 1. After exceeding the number 999 in "Compensation expectation (monthly / USD)" the error "undefined" is displayed and the information about the work does not save. It means that entering 1000 there will be a bug in the screenshot.

  • Bug 2. You can not save job information without uploading a PDF file. This means that even if you have already added a PDF file and it is saved in the work profile, you still have to choose it from the computer again and add to want to save the profile. Otherwise, the "No file found" error will be displayed. Also another screen in the screenshots section.

  • Bug 3. Can't remove resume file and can't save profile without it.

To Reproduce

  1. Go to "Manage Settings" and "Job Status" tab.
  • Bug 1: In the "Compensation expectation (monthly / USD)" field, enter 1000 or more choose pdf file and save.

  • Bug 2: In the "Compensation expectation (monthly / USD)" field, enter 999 or less and save without choose pdf file.

  • Bug 3: You can't delete resume and can't save profile without it.

Expected behavior

  • Bug 1: Ability to save amounts greater than 1000.
  • Bug 2: Ability to save a profile if you have already added a resume without having to re-select it.
  • Bug 3: The ability to remove resume and write without it.

Screenshots

Zrzut ekranu z 2019-07-25 10-16-49

Zrzut ekranu z 2019-07-25 10-29-31

Desktop (please complete the following information):

  • OS: Linux
  • Browser: Firefox Developer Edition
  • Browser Version: 69.0b7
@thelostone-mc thelostone-mc added the bug This is something that isn't working as intended. label Jul 25, 2019
@Adamits
Copy link
Contributor

Adamits commented Jul 27, 2019

Hi @ririen

I could not reproduce Bug 1. Had you added any other data to fields not displayed in the screenshot, that could be causing the issue? I am looking into bugs 2 and 3.

Adam

Adamits added a commit to Adamits/web that referenced this issue Jul 27, 2019
and only return an error if there is no resume file AND
there is not already a resume for the profile in

Fixes: Bug 2 of gitcoinco#4867
@oritwoen
Copy link
Contributor Author

@Adamits I have checked several browsers. It turns out that only in some of them the BUG 1 occurs.

  • Firefox Developer Edition – BUG
  • Firefox Normal – BUG
  • Chromium – BUG
  • Brave Dev Channel – Work
  • Google Chrome – Work
  • Opera Developer – Work

What the problem then? From what I noticed, the mentioned browsers that have a bug change the numbers above one thousand to those with a space or other special character. That is, the field is automatically converted 1000 to 1 000 - which causes a server-side problem and write error. I know that this is a problem with these special characters because by intercepting the request and removing the gap between 1 and zeros, everything has been written correctly.

@Adamits
Copy link
Contributor

Adamits commented Jul 29, 2019

@ririen Thanks a lot for looking into that. I will look into fixing this one when I have a chance. It looks like in chrome there is some feature that forces the value entered into the field to conform to a specification, so I think the best solution is to look into making it compatible with other browsers.

@Adamits
Copy link
Contributor

Adamits commented Jul 30, 2019

Could not reproduce yet on my mac (will try linux), but note that in safari I get the following error when I navigate to the job status page immediately when I navigate to it (it should have the Compensation field already populated):

Screen Shot 2019-07-29 at 10 14 55 PM

Adamits added a commit to Adamits/web that referenced this issue Jul 30, 2019
@Adamits
Copy link
Contributor

Adamits commented Jul 30, 2019

Ah - this is not browser specific, but rather because this javascript autoupdated the value to 'NaN', which saved.

I guess there are two issues here related to BUG 1:

  1. there should be some back-end validation so that a string cannot be saved to Profile.job_salary

  2. this front-end feature needs to work cross-browser

Adamits added a commit to Adamits/web that referenced this issue Aug 3, 2019
thelostone-mc pushed a commit that referenced this issue Aug 7, 2019
* Correctly check error_response status for integer 400
Fixes: Bug 2 of #4867

* Do not overwrite current Resume with None when saving.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This is something that isn't working as intended.
Projects
None yet
Development

No branches or pull requests

3 participants