appirio-tech / connect-app

Build your next project on Connect with the power of crowdsourcing
https://connect.topcoder.com
44 stars 139 forks source link

Creating customer success message page project intake submission #3949

Open RishiRajSahu opened 4 years ago

RishiRajSahu commented 4 years ago

Currently we show this message for successful submission of an intake request

Screenshot 2020-04-22 at 11 01 36 AM

We would need to make this success message configurable via template i.e. we would be having custom message for each kind of intake request.

Further requirements yet to be finalised, creating this issue to keep track of it.

fyi @acshields @vikasrohit

vikasrohit commented 4 years ago

@maxceem I think it should be customizable via metadata that we store at project template level.

maxceem commented 4 years ago

Sure @vikasrohit should we only customize this part of the message? Or any other part?

image

vikasrohit commented 4 years ago

Lets customize all three text, heading, subheading and description. All 3 would reside in same object against new key (say successPage) in our metadata.

maxceem commented 4 years ago

@vikasrohit the issue appears be to not that straightforward as it look like, though we can try to accomplish it in this release anyway.

We have to decide on the approach to handle this:

  1. This success page can accessed by the direct link https://connect.topcoder-dev.com/new-project/submitted/9166 - so we can access for any old project. If we want to keep supporting custom messages, then when we open this page we have to implement loading of the project (put it somewhere maybe in Redux store), so we can get its Project Template to show custom messages.

  2. Alternative solution would be somehow make this page not accessible by the direct link, and after pass the Project Template which has been used for creating project to this view. It's also surprisingly not so trivial as per current code as we remove all the project details as soon as we create it. So we might need to create some place in the Redux Store to hold the project object we've just created, to get the project template for it to display custom messages.

vikasrohit commented 4 years ago

I think we can go with option 2 and we even don't need to make the page not accessible. We can just show the default text (as it is right now) if don't have project passed as prop. Now coming to the second part where we are removing the project data from redux store, I am not able to recollect why we are doing that. But that gives me another thought that one of the other solution that came up in our internal discussions was to remove this submitted page and rather automatically take user to the project details page with a message there. If we go with that approach, I think we won't have problem of project loading into the state, right? Let me discuss that and come back to you.

maxceem commented 4 years ago

If we go with that approach, I think we won't have problem of project loading into the state, right?

Yep. On project details page we are loading project data into the state anyway, so we can just use it.

vikasrohit commented 4 years ago

Moving to next release as we don't yet have input from product manager.