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

Remove Deposit Requirement for Custom Role #504

Closed
wdprice opened this issue Sep 14, 2021 · 21 comments
Closed

Remove Deposit Requirement for Custom Role #504

wdprice opened this issue Sep 14, 2021 · 21 comments

Comments

@wdprice
Copy link

wdprice commented Sep 14, 2021

When a user submits a custom role request, do not include the cost of that resource in the deposit amount, and do not show it to the end user on the checkout page:

Revised design when the user has a normal role and custom role in the "cart": https://marvelapp.com/prototype/89c7i66/screen/82013112/handoff

Revised design when the user only has custom roles in the "cart": https://marvelapp.com/prototype/89c7i66/screen/82043216/handoff

@wdprice wdprice added the enhancement New feature or request label Sep 14, 2021
@wdprice wdprice added this to the TaaS Intake 1.3 milestone Sep 14, 2021
@nikolay83 nikolay83 self-assigned this Sep 15, 2021
@yoution
Copy link
Contributor

yoution commented Sep 15, 2021

@wdprice @nikolay83 we should set the price to 0, or we need to hide the custom role?
image
when can you provide the design only has custom roles?

@wdprice
Copy link
Author

wdprice commented Sep 15, 2021 via email

@sdgun
Copy link

sdgun commented Sep 16, 2021

@yoution If the user sets a title on the Job description page, we still show the deposit amount for the custom role on the checkout page.

2021-09-16_12-44-47.mp4

yoution added a commit to yoution/taas-app that referenced this issue Sep 16, 2021
nikolay83 added a commit that referenced this issue Sep 16, 2021
@sdgun
Copy link

sdgun commented Sep 17, 2021

Verified in Dev.

image

@sdgun
Copy link

sdgun commented Sep 17, 2021

@wdprice In the design I see this page https://marvelapp.com/prototype/89c7i66/screen/82043216/handoff which we haven't implemented yet. Are we going to have a separate ticket for this?

@wdprice
Copy link
Author

wdprice commented Sep 27, 2021

@sdgun - that design is the one was was marked as TBD in the initial design. I have updated the parent ticket.

CC: @nikolay83

@sdgun
Copy link

sdgun commented Sep 28, 2021

@wdprice Thanks. Just FYI, if the user only selects a custom role, there is no way to submit in the current implementation since the amount is 0.

@wdprice
Copy link
Author

wdprice commented Sep 29, 2021

The point of the new design is to allow them to submit custom role only.

@sdgun
Copy link

sdgun commented Sep 30, 2021

We have not implemented this part yet "Revised design when the user only has custom roles in the "cart":", so submitting only custom roles is not possible @nikolay83

@sdgun sdgun added the To Do label Oct 22, 2021
@sdgun
Copy link

sdgun commented Oct 22, 2021

@maxceem 'when the user(external) only has custom roles in the "cart"', they cannot proceed with the payments since we have changed the custom role amount to 0 in this ticket. The reason for this is we haven't implemented the second part of the requirement:

Revised design when the user only has custom roles in the "cart": https://marvelapp.com/prototype/89c7i66/screen/82043216/handoff

@maxceem
Copy link
Contributor

maxceem commented Oct 22, 2021

TODO:

  • If the total amount is 0 then show this page as per this design https://marvelapp.com/prototype/89c7i66/screen/82043216/handoff
  • we have to set agree checkmark to make Confirm button active
  • when we click Confirm show loading indicator somehow and send request to create a Team
  • after Team is created, redirect to the Team page
  • if error happens show red toastr with error and stay on the same page, so user can click Confirm again and try to send request again

@yoution would you like to handle this?
Payment: $40

@yoution
Copy link
Contributor

yoution commented Oct 22, 2021

@maxceem please assign to me

@maxceem maxceem assigned yoution and unassigned nikolay83 Oct 22, 2021
@maxceem
Copy link
Contributor

maxceem commented Oct 22, 2021

@yoution I've assigned it to you. Please, ignore that bot unassigned you.

@yoution
Copy link
Contributor

yoution commented Oct 22, 2021

@maxceem do we need to update the title?
image
Uploading image.png…

@maxceem
Copy link
Contributor

maxceem commented Oct 22, 2021

@yoution no, let's keep it the same as per design.

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

maxceem commented Oct 24, 2021

@yoution works as per local testing, passing for QA.

@sdgun this is ready for testing on DEV as per #504 (comment).

@sdgun
Copy link

sdgun commented Oct 25, 2021

@yoution @maxceem The title is different from the design, is this intentional?

image

image

@maxceem
Copy link
Contributor

maxceem commented Oct 25, 2021

@sdgun Hmm, it was like that before this fix already for 3 months. I'm not sure if this is intentional or no, but I guess if nobody asked to change during this time, it's better to keep the current text confirm your request.

@maxceem
Copy link
Contributor

maxceem commented Oct 25, 2021

@SathyaJayabal
Copy link
Collaborator

@sdgun ,
Screenshot 2021-10-26 at 12 22 38 PM
verified on safari/mac
Screenshot 2021-10-26 at 12 22 16 PM

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
@sdgun
Copy link

sdgun commented Oct 27, 2021

Verified in prod

Custom role with an existing role:

image

Only custom role:

image

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

6 participants