GoodDollar / GoodDAPP

GoodDollar.org Wallet is the simplest access point to Claim your daily G$. It Is based on web3 and React native web.
good-dapp.vercel.app
MIT License
102 stars 53 forks source link

Bug: hasJoinedInvites data-out-of-bounds #4109

Closed L03TJ3 closed 10 months ago

L03TJ3 commented 11 months ago

Description

error shown on console triggered by 'hasJoinedInvites'

Steps to reproduce

  1. go to dev.gooddollar.org
  2. switch chain from celo to fuse
  3. error appears in console

Which network did you use?

Supporting data

image

sirpy commented 11 months ago

@L03TJ3 this seems like an ABI issue, ie not being able to parse the blockchain data correctly. I suggest trying to upgrade goodprotocol package to the latest to see if that solves the issue. Is this reproducible on qa/dev?

L03TJ3 commented 11 months ago

@vldkh can you fill in the steps to reproduce or if not reproducible easily the steps you did before this got triggered e.g: was on initially loading dashboard / did you do claiming before / interacted with rewards screen in any way?

@sirpy You mean the one you just released? as we were already on .13

sirpy commented 11 months ago

yes

patpedrosa commented 11 months ago

@vldkhh Is this happening on prod?

vldkhh commented 11 months ago

@patpedrosa for me no, maybe it is a different case specific to the dev account

sirpy commented 10 months ago

@L03TJ3 was the package upgraded? Also could be we need to upgrade the invite contracts for dev/qa env. @vldkhh please let us know if this happens on: dev/qa celo dev/qa fuse

L03TJ3 commented 10 months ago

@sirpy yes

sirpy commented 10 months ago

@L03TJ3 yes and it still happens? or yes and we haven't tested yet

L03TJ3 commented 10 months ago

Apologies. @sirpy . Yes this was tested on the last version. But got to say this was the latest version when this was discussed. .14

Not retested with latest .15

L03TJ3 commented 10 months ago

now bumped to .15

@vldkhh could you retest this today?