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

enables sending of kudos on the xdai network right from the browser. #7737

Merged
merged 3 commits into from
Dec 22, 2020

Conversation

owocki
Copy link
Contributor

@owocki owocki commented Oct 21, 2020

Description

enables sending of kudos on the xdai network right from the browser.

Refers/Fixes

broken UX on the live site

Testing

tested locally works fine.

the only wrinkle is that the kudos price is in finney (based on ETH). we could perhaps transpose that to an xDAI price with conversion rates.. but thats added complexity. i kinda feel like its not worth the effort since kudos isnt a large forward development priority for us. but am open to feedback if not.

@owocki owocki changed the base branch from master to stable October 21, 2020 22:05
@thelostone-mc
Copy link
Member

mind if i get this into master? @owocki

@owocki
Copy link
Contributor Author

owocki commented Nov 19, 2020

sure!

@owocki owocki changed the base branch from stable to master November 19, 2020 14:02
@owocki
Copy link
Contributor Author

owocki commented Nov 19, 2020

pr updated

@owocki owocki mentioned this pull request Nov 24, 2020
@owocki
Copy link
Contributor Author

owocki commented Nov 24, 2020

@thelostone-mc can i merge for #7919

@owocki
Copy link
Contributor Author

owocki commented Dec 21, 2020

can i merge this?

@danlipert danlipert merged commit 9c1fcff into master Dec 22, 2020
@ciprianiacobescu
Copy link

Hi all!

I got the following error "You are not on the right web3 network. Please switch to xdai". I was already on xdai network! Tried a CTRL+R, but still received the same message. I'm using Metamask with a Ledger wallet. I've used the xdai network to get some honey, just before this...

@owocki
Copy link
Contributor Author

owocki commented Jan 3, 2021

its not deployed yet. @danlipert can let us know when the deploy is happening

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.

4 participants