New UI for Investor to provide clarity at the time of Investing #435
Conversation
There was a problem hiding this comment.
it seems this icon i smaller then the others?
There was a problem hiding this comment.
I missed removing this SVG in the last commit, not using this anywhere in the code
src/Angor/Client/Pages/Invest.razor
Outdated
| @@ -626,6 +745,52 @@ else | |||
| { | |||
| await RefreshSignatures(); | |||
| } | |||
| RefreshWizardState(); | |||
There was a problem hiding this comment.
this methods should go inside the if above no?
src/Angor/Client/Pages/Invest.razor
Outdated
| @@ -645,6 +810,51 @@ else | |||
| passwordComponent.ShowPassword(ScanForPendingSignatures); | |||
| } | |||
| } | |||
| RefreshWizardState(); | |||
There was a problem hiding this comment.
this 3 methods are called twice once here and once in the render
|
|
|
The text is too light grey |
|
Where does this text come from? |
|
perhaps the text box on the popup should say "request to invest" or "send request" |
I can remove this If you say or replace it with something else, this was just to balance out with the left side, now there is much empty space in the right side |
|
Hi Dan, you tell me which browser you are using? I tried this on Safari & Chrome, looking fine on my side |
src/Angor/Client/Pages/Invest.razor
Outdated
| </div> | ||
| } | ||
| <p class="agreement-text text-center"> | ||
| By investing, you agree to the project terms and recovery agreement |
There was a problem hiding this comment.
I think just remove this text, a user may get confused and wonder where re those project terms?
There was a problem hiding this comment.
or you can put there the penalty days that a user will wait if they bail out
| } | ||
|
|
||
| private async Task ScanforProjectStats() | ||
| { |
There was a problem hiding this comment.
do we need to call the indexer every time? it seems to me that once we call it then we don't need to do it again? we already have the stats.
There was a problem hiding this comment.
I called the Indexer to get the TotalRaised amount, which is used to show the % in the progress bar, the other time it is called is to know if the investor is already invested at the time of investing
src/Angor/Client/Pages/Invest.razor
Outdated
| @@ -1090,6 +1300,7 @@ else | |||
| investorProject.CompleteProjectInvestment(signedTransaction.Transaction); | |||
|
|
|||
| storage.UpdateInvestmentProject(investorProject); | |||
| invested = true; | |||
There was a problem hiding this comment.
this is to change the page view but then we anyway redirect way so it is a bit pointless
| @@ -607,7 +607,7 @@ else | |||
| </button> | |||
| </div> | |||
| <p class="agreement-text text-center"> | |||
| By investing, you agree to the project terms and recovery agreement | |||
| By investing, you acknowledge that early withdrawal requires a @project.ProjectInfo.PenaltyDays day waiting period | |||
| @@ -1228,6 +1228,7 @@ else | |||
| if (res.ProjectIdentifier == investorProject.SignaturesInfo?.ProjectIdentifier) | |||
| { | |||
| investorProject.SignaturesInfo.Signatures = res.Signatures; | |||
| RefreshWizardState(); | |||
There was a problem hiding this comment.
one more change this is not a wizard it is a progress
|
I wonder now why do we have two cards that say waiting for signatures? |
|
I have made a planka card for this issue, will make a separate PR for this 👍 |
BEFORE:
AFTER:
Mobile View