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

fix[config]: config for cyber #89

Closed

Conversation

dimakorzhovnik
Copy link
Contributor

when creating tx keplr was white, after changed feeCurrencies for cyber in the config, all right

- changed feeCurrencies for cyber in config
@dimakorzhovnik dimakorzhovnik mentioned this pull request Apr 21, 2021
@Thunnini
Copy link
Member

What version of Keplr are you using? Also, could you show me the error message that shows up in the console for the blank page when you send the tx?

Cyber's feeCurrencies being empty is intentional. Cyber doesn't use fees, so it is left empty–and Keplr is currently designed where it works without fees when feeCurrencies is empty.

@dimakorzhovnik
Copy link
Contributor Author

dimakorzhovnik commented Apr 21, 2021

v0.8.5
2021-04-21 19 31 13

I have cloned repository and change config, run build keplr and used local build, wasn't mistakes

Sometimes need to run chrome://flags/#enable-webassembly for os windows and linux

@cyborgshead
Copy link
Contributor

@Thunnini Hello. Please, merge this issue because keplr not work at all with cyber.page in latest releases of keplr if fee not set.

Thunnini added a commit that referenced this pull request Jun 1, 2021
@Thunnini
Copy link
Member

Thunnini commented Jun 1, 2021

Cyber not having a fee currency is intentional. Because the Cyber chain doesn’t need tx fees, there is not fee currency–which will hide the fee buttons. However, I think there may have been issues handling a signature requested from a webpage. I believe the commit above should solve the issue, and will be included in the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants