Marcelo Morgado

Results 49 comments of Marcelo Morgado

Use ENS to set the beneficiary address. @pcowgill commented: With ENS the main thing that’s important in the demo app is easy pasting of copied text, and client-side form validation...

I've clean up my app data and followed the below steps trying to reproduce these issues: - Signup - Buying a land (name: TOP) - It's was added to MyLandScreen...

Did you do this test using `develop` branch or the PR branch? Is this related to https://github.com/tasitlabs/tasit/pull/260#issuecomment-481261606 ?

> > Do a revert operation > > When you say revert do you mean reverse? "revert" means "undo" to me. Oh, sorry. I've meant "reverse list".

Is he a new tester? Isn't an expected reset account behavior?

@pcowgill commented: > I think React context API is worth considering in the medium term. Refs: https://github.com/tasitlabs/tasit/pull/93#issuecomment-462464898

Seems that [redux docs](https://redux.js.org/advanced/async-actions) recommends keep `actions` as plain functions and use middleware (thunk/saga) to handle with fetching data. What do you think? Refs: https://github.com/tasitlabs/tasit/pull/93#discussion_r255698391

**TODO** Move functions that interact with blockchain to actions / middlewares.

@pcowgill commented: > I'm wondering whether the await style still makes sense for higher-level functions that wrap Ethereum interactions. I think it doesn't but we'll want to set best practices...