MultiCurrencyWallet icon indicating copy to clipboard operation
MultiCurrencyWallet copied to clipboard

fix share button

Open shendel opened this issue 1 year ago • 4 comments

Checklist

  • [ ] I have read the CONTRIBUTING guide
  • [ ] Good naming (as clear and simple as possible)
  • [ ] Correct behavior if external API endpoints are down, return 404, 504 (or no answer), 401 errors (ddos simulation)
  • [ ] I tested desktop/mobile resolution
  • [ ] I tested light/dark theme
  • [ ] I tested different languages
  • [ ] I checked the functionality once again (AFFECT MONEY)
  • [ ] I checked the work on the Testnet
  • [ ] I checked the work on the Mainnet
  • [ ] I checked the work in the plugin
  • [ ] I checked the PR once again

Tests

Please start auto tests as follows:

  • add a label swap test to start swap tests
  • add a label withdraw test to start withdraw tests

You can skip these tests if you completely sure that your changes aren't related to this functional

Original issue

https://github.com/swaponline/MultiCurrencyWallet/issues/5106

Video / screenshot proof

shendel avatar Aug 18 '22 21:08 shendel

А что с тестами? до этого работали вроде, сейчас вообще все повалились...

VitaliyShulik avatar Aug 19 '22 05:08 VitaliyShulik

А что с тестами? до этого работали вроде, сейчас вообще все повалились...

надо увеличить таймаут на восстановление кошелька я думаю. с новыми либами долго идет первая инициализация

shendel avatar Aug 24 '22 11:08 shendel

если тесты не работают просто проверить че они проверяют руками и все

noxonsu avatar Aug 31 '22 20:08 noxonsu

Да, там можно изменить дебагинг в утилитах на true и будет открываться окно браузера, можешь посмотреть локально что там долго грузится...

https://github.com/swaponline/MultiCurrencyWallet/blob/022195171ee54124cc429e143c8c56ec7f65ff4b/tests/e2e/utils.ts#L17-L19

VitaliyShulik avatar Sep 01 '22 04:09 VitaliyShulik