Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

User Facing Light Client Issues #298

Closed
MrCleanerPickerUpper opened this issue Aug 31, 2019 · 6 comments
Closed

User Facing Light Client Issues #298

MrCleanerPickerUpper opened this issue Aug 31, 2019 · 6 comments
Labels
bug 🕷️ Something isn't working dApp 📱

Comments

@MrCleanerPickerUpper
Copy link

MrCleanerPickerUpper commented Aug 31, 2019

Issues found while using light client:

After connecting to Hub (0x5257964Ef9b81fba7276AF2a97c111AaD7B840D6) it doesnt show how much WETH I have available to open a channel, it only shows available WETH once ive typed in a number.

After typing in a number I have to put a whole number (1,2,3,4...) before it allows me to put a decimal place. I.e. I cannot just put .5, i have to put 3.5 then delete the 3.

When opening a channel, there is a step guy on top showing the stages. The image is obscured, each process has a strange diamond/square on top to make the steps not readable.

raidengraphic

Unable to open channel: Set total Deposit Failed

error

When retrying it goes straight to "oops something went wrong, channel open failed"

@christianbrb
Copy link
Contributor

@MrCleanerPickerUpper Thanks for testing the light client :-)

For the first two paragraphs, I have created separate issues (see #299 and #300).

Could you provide us information

  • which browser you are using
  • the error shown in the browser console logs (if you are familiar with it)

@kelsos
Copy link
Contributor

kelsos commented Sep 2, 2019

@MrCleanerPickerUpper The current version uses the Red Eyes contracts. The Red Eyes contracts unfortunately only allow up to 0.15 as a deposit and everything bigger will actually fail. We will start a discussion to integrate the limit checking in the newer version.

In regards to the stepper, I have seen this in a few browsers and we should probably remove it temporarily until we find a cross-browser compatible way to implement that doesn't break. @roy-castrillo.

@roy-castrillo
Copy link

roy-castrillo commented Sep 3, 2019

@kelsos remove the stepper completely or leave it, but remove the shapes behind them and highlight blue only the current step? See the example:

Screenshot 2019-09-03 at 13 57 29

@kelsos
Copy link
Contributor

kelsos commented Sep 3, 2019 via email

@christianbrb christianbrb added the bug 🕷️ Something isn't working label Sep 5, 2019
@christianbrb christianbrb added this to the Product Backlog milestone Sep 5, 2019
@christianbrb christianbrb changed the title User Facing Light Client Issues User Facing Light Client Issues - Redesign stepper Sep 5, 2019
@christianbrb
Copy link
Contributor

christianbrb commented Sep 5, 2019

Acceptance criteria
-The best simple solution is implemented

Tasks

@christianbrb christianbrb changed the title User Facing Light Client Issues - Redesign stepper User Facing Light Client Issues - Fix stepper Sep 5, 2019
@christianbrb christianbrb added the 3 label Sep 5, 2019
@christianbrb christianbrb removed this from the Product Backlog milestone Sep 5, 2019
@christianbrb christianbrb changed the title User Facing Light Client Issues - Fix stepper User Facing Light Client Issues Sep 5, 2019
@christianbrb christianbrb removed the 3 label Sep 5, 2019
@christianbrb
Copy link
Contributor

@MrCleanerPickerUpper Thanks again for your feedback

We have created two additional issues regarding your feedback:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🕷️ Something isn't working dApp 📱
Projects
None yet
Development

No branches or pull requests

4 participants