Skip to content
This repository was archived by the owner on Mar 13, 2025. It is now read-only.

Don't display a role without a global rate #462

Closed
nikolay83 opened this issue Aug 11, 2021 · 11 comments
Closed

Don't display a role without a global rate #462

nikolay83 opened this issue Aug 11, 2021 · 11 comments

Comments

@nikolay83
Copy link
Collaborator

Is there is a role in the database that has a global rate (the full-time one) empty or null, then this role shouldn't

  • be displayed in the Roles list
  • be found when searching via Skills or Job Description flow
yoution added a commit to yoution/taas-app that referenced this issue Aug 12, 2021
@yoution
Copy link
Contributor

yoution commented Aug 12, 2021

@nikolay83 please review
topcoder-platform/taas-apis#479
#462

nikolay83 added a commit that referenced this issue Aug 12, 2021
@sdgun
Copy link

sdgun commented Aug 12, 2021

@nikolay83 @wdprice Should this only be applied to Global rate (full time)? In my comment on issue 434, it's missing part time global rate which cause the amount to be displayed as 'NaN'

@wdprice
Copy link

wdprice commented Aug 12, 2021

Ah OK, so we need Global Part time rates added? That is easy enough to fix - it is a data issue.

Looks like we cannot edit the Custom Role data in the admin console, so I'll specify here:
30 hour rate = Global Rate * .75
20 hour rate = Global Rate * .5

@nikolay83
Copy link
Collaborator Author

@yoution
2 extra requirements:

  1. global20 and global30 rates also shouldn't be empty or null. So display only the roles that have all 3 global rates not empty.
  2. display the part-time rate for the custom role (the full-time rate is in config) as
    30 hour rate = Global Rate * .75
    20 hour rate = Global Rate * .5

@yoution
Copy link
Contributor

yoution commented Aug 13, 2021

@nikolay83 the full-time rate is in config, does it already exist?I can not found the config value

yoution added a commit to yoution/taas-app that referenced this issue Aug 13, 2021
yoution added a commit to yoution/taas-app that referenced this issue Aug 13, 2021
nikolay83 added a commit that referenced this issue Aug 13, 2021
nikolay83 added a commit to topcoder-platform/taas-apis that referenced this issue Aug 13, 2021
@sdgun
Copy link

sdgun commented Aug 13, 2021

@yoution The rate is set incorrectly. For 30 hrs it should be Global Rate * .75 and for 20hrs Global Rate * .5, but it is vice versa now.

2021-08-13_11-32-10.mp4

@yoution
Copy link
Contributor

yoution commented Aug 13, 2021

@sdgun I will check

@nikolay83
Copy link
Collaborator Author

@yoution
customRole.rates[0].rate20Global = customRole.rates[0].global * 0.75
customRole.rates[0].rate30Global = customRole.rates[0].global * 0.5

Should be vice versa, 0.75 for 30 hours

yoution added a commit to yoution/taas-apis that referenced this issue Aug 13, 2021
@yoution
Copy link
Contributor

yoution commented Aug 13, 2021

@nikolay83 please review

@sdgun
Copy link

sdgun commented Aug 13, 2021

Verified in Dev, fixed.

@sdgun
Copy link

sdgun commented Aug 16, 2021

Verified in production.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants