Paratii-Video / paratii-js

JS library for clients to interact with deployed contracts, with wallet functionality, access to blockchain data and convenience functions.
GNU General Public License v3.0
35 stars 10 forks source link

the paratii.users.create promise is taking ~30 seconds or more to resolve #244

Open jellegerbrandy opened 6 years ago

jellegerbrandy commented 6 years ago

This is what @bent0b0x reports: https://github.com/Paratii-Video/paratii-portal/issues/582#issuecomment-390404155

bent0b0x commented 6 years ago

Is it possible for someone to look into this? I see issues w/ user creation/lookup consistently locally and it makes it quite difficult to develop new profile features, specifically this one. Perhaps this is only an issue on my machine but not sure why it would be.

Thanks in advance!

jellegerbrandy commented 6 years ago

@bent0b0x , I cannot reproduce this.. you see this in the tests as well?

bent0b0x commented 6 years ago

How have you tried to repro it? There is no blocking in the UI but if you put a .then after paratii.users.create the callback was not hit for 30 seconds (at least in my experience, and at least locally).

This leads to us not being able to fetch user data until this time span has elapsed

jellegerbrandy commented 6 years ago

I call this in devtools in the browser and it returns immediately (<.5 secs I think):

paratii.eth.users.create({id: '0xCbe4f07b343171ac37055B25a5266f48f6945b7d', name: 'jelle', ipfsData: 'xxxxx'}).then(() => { console.log('done') })

On Fri, May 25, 2018 at 1:56 PM Ben notifications@github.com wrote:

How have you tried to repro it? There is no blocking in the UI but if you put a .then after paratii.users.create the callback was not hit for 30 seconds (at least in my experience, and at least locally).

This leads to us not being able to fetch user data until this time span has elapsed

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/Paratii-Video/paratii-js/issues/244#issuecomment-392030605, or mute the thread https://github.com/notifications/unsubscribe-auth/ABPuPd8RJBYS8gLg8iHeTtefSEWKwu0qks5t1_FkgaJpZM4UF1Qy .

bent0b0x commented 6 years ago

Connected to local dB? I will try again today, perhaps it was a blip but I did have this problem on multiple days

jellegerbrandy commented 6 years ago

Mm, I actually did try the wrong function call (it should be this). Runing agians local db, latest dev version:

 paratii.users.create({id:

'0xCbe4f07b343171ac37055B25a5266f48f6945b7d', name: 'jelle', ipfsData: 'xxxxx'}).then(() => { console.log('done') })

On Fri, May 25, 2018 at 3:29 PM Ben notifications@github.com wrote:

Connected to local dB? I will try again today, perhaps it was a blip but I did have this problem on multiple days

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/Paratii-Video/paratii-js/issues/244#issuecomment-392058104, or mute the thread https://github.com/notifications/unsubscribe-auth/ABPuPUFj_kwKufD5qYWC6MkvtmZLRLf8ks5t2Ac-gaJpZM4UF1Qy .