Syrius v0.2.0-rc.2 #3

Development
1

This post will be used to collect issues for test round #3.

Download: Syrius v0.2.0-rc.2

Read this post for more information about how to test.

1 Like
2

It wasn’t my best start, failed at the second row. See issue. Turns out it is not a big deal, just a chain id mismatch, but this took me 20 minutes to figure out. I realized what was going on thanks to this detailed notification, when trying to fuse plasma to the original wallet from another one:

image
image796Ă—110 3.33 KB

Would it be possible to enhance this one with a similar error message, in this particular case?:

image

Edit: I guess the issue is not the notification above, but that it’s silently failing without an “Error while generating Plasma” notification error.

Edit2: I also wanted to mention a small UI/UX improvement. During the Node Management on boarding flow, the default window size does not accommodate enough space to show the “Wallet options” and “Continue” button sections, and there is no scroll bar indicating the user there is more content hidden from view.

Anyway… will proceed to try to get more PASSes!

3

Thank you for your submission.

Coincidentally, two issues have been recently picked up that are somewhat related to your issue. These are:

I believe this will solve the small UI/UX improvement you mentioned.

Concerning the issue, I was not able to reproduce the problem exactly as mentioned. The plasma generation loop is by design, because the auto-receiver will try to receive the tx until success.

The difference is that I’m getting an error notification after the plasma generation notification wereas in your case you keep getting only plasma generation notifications. This is the part I cannot seem to reproduce.

errors
errors1920Ă—1040 38.5 KB

4

That’s interesting. Ok, I will try again in another machine or in another OS and report back or try to reproduce it.

5

Should the client chain id be automatically set by syrius after the user clicks “proceed anyway” ?

In Metamask, it’s expected behavior when I get a popup to “change networks/RPC” and I accept, that it automatically changes the chain id, explorer, and everything MM requires to function properly. Should this not be the case in syrius?

Maybe a slight modification of the Popup that reflects the changes that will happen when “proceeding”.

6

Currently the chain id mismatch only warns the user. It is assumed the user knows what he’s doing.
Metamask works differently because it has predefined networks to choose from. They have to be configured explicitly beforehand.
Syrius allows both the node url and chain id to be used interchangeable. Which is not necessarily a good thing.

The chain id and node url are to my knowledge bound to eachother. Meaning you cannot connect the hc1 node with chain id other than 1. While other nodes are specifically meant for testing. So it makes sense to combine both the node url and chain id in one single configration.

One other limitation of the node management screen while onboarding is that you cannot specify the chain id. So for example, if you want to create a new wallet and connect a testnet. You first have to connect to the testnet node using chain id 1 while onboarding and later goto settings and change the chain id.

In general changing the chain id only has to done for testnets or devnets. The average user should not directly be bothered.

I think the node management needs to be redesigned, but later for another future release.

7

These issues have been resolved in the following PRs:

A new release candidate will be made available.