Improve the interface of useWallet hook

[emanueltimlopez]

Its hard to know what hook exposes.

I think we can declare an interface in the provider and only provide the connection properties when connection is success.

[Dhaiwat10]

@emanueltimlopez I think we still want to keep the current connection API. The connection object being the single source of truth & reactive store is something I really like. Let me know if you have other ideas and why, though! I'm open to new ideas.

The issue is with all those undefined and null cases. You'll often come across context?.something in the code which is not ideal. We want to clean that up.

[emanueltimlopez]

i undertand, i think its a lot better now

BTW i dont have permission to push my branch @Dhaiwat10

[etr2460]

@emanueltimlopez if you don't have write permission to the repo, you can fork it, push a branch to your fork, and then open the PR here

[Dhaiwat10]

@emanueltimlopez I have made some changes to the useWallet hook via #74. Please have a look at that PR and keep those in mind!

[Dhaiwat10]

@allcontributors add @emanueltimlopez for code! <3

[allcontributors]

@Dhaiwat10

I've put up a pull request to add @emanueltimlopez! 🎉