Support » Plugin: EthPress - Web3 Login » Wallet connect trust wallet

  • Resolved tutbozero

    (@tutbozero)


    Trustwallet has updated to v1 3 days ago, this has broke the signing / logging in.

    I’ve tried to read through the code but I’m stuck, any idea how to fix this?

    A speedy response Would be appreciated thanks

Viewing 12 replies - 16 through 27 (of 27 total)
  • Thread Starter tutbozero

    (@tutbozero)

    Hi Lynn, It looks pretty good, we have not had any issues so far overnight testing, in fact its more reliable imo.

    There is an issue on iphones that existed before where you have to sign twice because the first time it takes you to trust wallets website for some reason (one of the things I said i would like to simplify)

    things id like to fix.
    – iphone double signin
    – removal of eth table in sql on uninstall of plugin (it should remain)
    – option to change from eth display/signin to another token 😛

    Do you take BNB to your wallet, and can you confirm it and ill pop some beer tokens in it?

    Plugin Author lynn999

    (@lynn999)

    Cool,

    Does this double login happen on example.walletconnect.org ?

    Yea the table should stay, I agree, I will change that in the next release.

    Which display are you talking about?

    0x106417F7265E15C1aAe52F76809F171578e982a9 is my eth address (also listed on the readme), yeah you can send BNB and bsc tokens to it as well (as far as I’m aware), all good.

    I will update the wp.org listing tomorrow, hopefully.

    Thread Starter tutbozero

    (@tutbozero)

    Which display are you talking about?
    I am talking about the wallet connect when it signs it shows eth in the wallet when it signs, its not a big issue just we are on the bnb network normally, just would be nice to see bnb or anything else :P.
    I did see a part in the php where it says coin 60 I think, i wasnt sure if changing that would change the wallet connect show.

    Does this double login happen on example.walletconnect.org ?
    No it works first time, I noticed in the code there is a link to trustwallet website if the wallet isnt installed, i wondered about removing that.

    Thanks for your awesome assistance.

    Plugin Author lynn999

    (@lynn999)

    Where do you see this link? Probably it’s causing it, but I don’t see such a thing in my code. Sounds like it’s in walletconnect code..?

    About the display, dunno if it is doable, but either way I’m atm too busy to make it. New maintainer will be joining us soon, perhaps they will have time for that. I like the idea tho!

    Thread Starter tutbozero

    (@tutbozero)

    ill have a look today, i went through every file the other day and noticed it.

    Everything is working well still so looks like we are good, we have sent you the drink to your wallet for helping out so fast, thanks again!.

    Plugin Author lynn999

    (@lynn999)

    Could you tell me what version of WordPress you’re running, so I can update the “latest tested version” tag accurately?

    Thread Starter tutbozero

    (@tutbozero)

    Current version: 5.8
    Last checked on August 6, 2021

    Plugin Author lynn999

    (@lynn999)

    Alright, thanks.

    I’ve updated the development version, it can be downloaded from https://wordpress.org/plugins/ethpress/advanced/ . I will try get another person to test it before I put it out there as stable, just to be on the safe side.

    And big thanks for the tip!

    Plugin Author lynn999

    (@lynn999)

    Also, I would appreciate if you tried that development version too, if it’s not too much trouble, just so that I know that it’s not broken. It should essentially be the same as your current install.

    I didn’t fix the double login thing now, I reckon that will take some investigating. I did fix the table uninstall thing.

    Thread Starter tutbozero

    (@tutbozero)

    I copied it over on the test site, will let you know tomorrow.

    Plugin Author lynn999

    (@lynn999)

    Any problems? I’ll update the stable tag if not. Nobody else showed up for testing so far (as expected), so I’ll get bug reports — if any — post-update, which should be fine.

    Plugin Author lynn999

    (@lynn999)

    I’ve updated the stable tag. I hope I didn’t mess up the deploy, been a while since I’ve used SVN last… Should be good, I’m marking this ticket as resolved.

    Ah yeah, the double login problem is still there I suppose, I’ll open another ticket for that.

    • This reply was modified 1 year, 3 months ago by lynn999.
Viewing 12 replies - 16 through 27 (of 27 total)
  • The topic ‘Wallet connect trust wallet’ is closed to new replies.