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

[$30] The Deposit amounts are not calculated for newly added roles #525

Closed
sdgun opened this issue Oct 25, 2021 · 16 comments
Closed

[$30] The Deposit amounts are not calculated for newly added roles #525

sdgun opened this issue Oct 25, 2021 · 16 comments
Assignees
Labels
ACCEPTED bug Something isn't working Feedback The issue hasn't been fixed completely and some feedback has been provided. P1 PAID QA Pass on Dev QA Pass on Prod Role-Matching-Intake

Comments

@sdgun
Copy link

sdgun commented Oct 25, 2021

Steps
Add a new role from admin app with rates
Go to Intake app and select this role(log in from an external user)
Proceed to checkout page and see the deposit amounts

Actual behaviour
The Deposit amounts are not calculated for newly added roles

Expected behavior
The Deposit amounts should be calculated for newly added roles too

image

image

image

CC @maxceem

@sdgun sdgun added bug Something isn't working P1 Role-Matching-Intake labels Oct 25, 2021
@maxceem maxceem changed the title The Deposit amounts are not calculated for newly added roles [$30] The Deposit amounts are not calculated for newly added roles Oct 25, 2021
@maxceem maxceem added this to the TaaS Intake 1.3 milestone Oct 25, 2021
@maxceem
Copy link
Contributor

maxceem commented Oct 25, 2021

@yoution would you like to handle this?

@yoution
Copy link
Contributor

yoution commented Oct 25, 2021

@maxceem please assign to me

yoution added a commit to yoution/taas-app that referenced this issue Oct 25, 2021
yoution added a commit to yoution/taas-app that referenced this issue Oct 26, 2021
maxceem added a commit that referenced this issue Oct 26, 2021
@maxceem
Copy link
Contributor

maxceem commented Oct 26, 2021

@sdgun this was fixed on DEV and ready for QA.

PS: basically, this is a side effect of the fix for #504 so these kind of the same issue.

@maxceem
Copy link
Contributor

maxceem commented Oct 26, 2021

@yoution found one issue of this solution during QA #529.

@maxceem maxceem added Feedback The issue hasn't been fixed completely and some feedback has been provided. QA Fail in Dev and removed Ready for DEV QA ACCEPTED labels Oct 26, 2021
@yoution
Copy link
Contributor

yoution commented Oct 26, 2021

@maxceem I think I can not reproduce the issue
image

@maxceem
Copy link
Contributor

maxceem commented Oct 26, 2021

@yoution did you try clicking pay $1000? because the issue happens after we try to pay.

@yoution
Copy link
Contributor

yoution commented Oct 26, 2021

@maxceem the payment amount is 1000? what do you mean?
image

@sdgun
Copy link
Author

sdgun commented Oct 26, 2021

@yoution This issue only happens when you have a custom role(no matching role in our app) is submitted. And in both flows(external/internal users)

You can find the details in #529

@yoution
Copy link
Contributor

yoution commented Oct 26, 2021

@maxceem please ignore, I know the issue

@maxceem
Copy link
Contributor

maxceem commented Oct 26, 2021

UPDATE: please ignore this report

@yoution also, I'm not sure why, but it started showing me price when I create a customer role using job description:

image

Steps to reproduce:

  1. Clear all roles in storage so there are no roles saved.

  2. Click "Create New Team"

  3. Select "Describe the Work"

  4. Enter title: "test", and text: angular jsp jfs node

    image

  5. Choose "jsp" only

    image

  6. Click "continue", "continue"

  7. Enter Team Name and click Submit:

    image

  8. It shows custom role prize $1200, but as I understand it has to show 0 in this case:

    image

@maxceem
Copy link
Contributor

maxceem commented Oct 26, 2021

@yoution please ignore issue report above #525 (comment).

I guess it was cache issue, it works good in the incognito mode.

@yoution
Copy link
Contributor

yoution commented Oct 26, 2021

@maxceem please review

@maxceem
Copy link
Contributor

maxceem commented Oct 26, 2021

@sdgun the #529 was fixed and ready for QA.

@sdgun
Copy link
Author

sdgun commented Oct 27, 2021

This issue is fixed, amounts are calculated accurately.

image

@maxceem
Copy link
Contributor

maxceem commented Oct 27, 2021

@maxceem maxceem added the PAID label Oct 27, 2021
@sdgun
Copy link
Author

sdgun commented Oct 27, 2021

Verified in Prod

image

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ACCEPTED bug Something isn't working Feedback The issue hasn't been fixed completely and some feedback has been provided. P1 PAID QA Pass on Dev QA Pass on Prod Role-Matching-Intake
Projects
None yet
Development

No branches or pull requests

3 participants