search

poanetwork / token-wizard

(Discontinued) TokenWizard is an DApp to create and manage crowdsale and token contracts using a simple UI
MIT License
386 stars 215 forks source link

(Bug) Contribution page:countdown timer has incorrect status after finalization #1029

Closed dennis00010011b closed 6 years ago

dennis00010011b commented 6 years ago

Steps:

  1. Create crowdsale with any parameter, ex. :Minted, 1 tier, duration 2 min, minCap
  2. Wait until crowdsale ended
  3. As an owner open manage page and finalize
  4. Open contribution page and observe countdown timer.

Expected result: -timer's status is "CROWDSALE HAS BEEN FINALIZED""

Actual result: -timer's status is "CROWDSALE HAS ENDED"

If you are reporting a problem with Token Wizard, please include the following information:

Which network did you use? (Mainnet, Kovan, Rinkeby, etc.)

localhost

If you were able to create it, what is the URL of your crowdsale?

n/a

Do you have screenshots showing the problem?

screen shot 2018-07-11 at 00 08 37

Do you see errors in the dev console? If yes, please include a screenshot

No

mariano-aguero commented 6 years ago

@dennis00010011b mmm I tried to reproduce the problem without success Can you confirm if it keeps happening? Maybe I missed some step Thanks!

screenshot_20180711_153024

Step by step in a video https://www.useloom.com/share/ce3a2ae15b7541d59a3cf6f18b4902f8

dennis00010011b commented 6 years ago

Not reproducible in branch 2.0

dennis00010011b commented 6 years ago

It is again reproducible in branch 2.0 http://localhost:3000/contribute?addr=0xa82dd65d279a54be6fed602d47c5941a7b0c621e&networkID=3 Proxy address 0xa82dd65d279a54be6fed602d47c5941a7b0c621e , Ropsten

mariano-aguero commented 6 years ago

Hi @dennis00010011b , It seems like the problem was solved. The merge of the PR for the issue 1030 was the cause.

Manage page, where the crowdsale was ended and finalized screenshot_20180718_091718

screenshot_20180718_091703

Contribution page, where the crowdsale shows the expected result screenshot_20180718_091732

Can you confirm is solved? Thanks in advance

dennis00010011b commented 6 years ago

@mariano-aguero It still reproducible for Dutch strategy. I think it can be fixed together with https://github.com/poanetwork/token-wizard/issues/1049