Fix/metamask connection

[octavioamu]

Revert changes from a2e3804#diff-9bd96a6d2da4e99c0d1b9e27c7ead8b9R461

since this break metamask in any browser and for opera last version the web3 injection is minimal just to see the address, so need to load web3.js. Also there are a few diff
when sending a tx need go specify the field "from" (metamask didn't need that)
Also wallet api in opera works as others browser apis need the ask for permission to access or is not going to work (like metamask.enable() ) in opera need to run web3.currentProvider.enable() to and approve the permission, but there is not injection to expose the wallet without loading web3.js

This PR just fix the isBountyOwner returning always false.

Description

Checklist

• Read and conforms to the contributor guidelines
• Read and conforms to the style guidelines
• Linter status: 100% pass
• Added relevant tests
• Tests pass and code coverage has not decreased
• Changes don't break existing behavior
• Commit message follows commit guidelines
• Code documentation
• Completed: Code review by core team and any requested changes

Affected core subsystem(s)

Refers/Fixes

Testing and Sign-off

Contributor

• Read and followed the Contributor Guidelines
• Tested all changes locally
• Verified existing functionality
• Ran make test and everything passed!

Reviewer

• Affirm contributor guidelines have been followed and requested changes made
• CI tests and linting pass
• No conflicts (migrations, files, etc)
• Regression tested against staging or local deployment

Funder

• Validated requested changes were made to specification
• Bounty payout released to the contributor

[thelostone-mc]

if (typeof web3 == 'undefined' || !web3.eth ||...) {

[codecov]

Codecov Report

Merging #4211 into stable will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           stable    #4211   +/-   ##
=======================================
  Coverage   30.28%   30.28%           
=======================================
  Files         209      209           
  Lines       16594    16594           
  Branches     2220     2220           
=======================================
  Hits         5025     5025           
  Misses      11389    11389           
  Partials      180      180

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a2e3804...b295b53. Read the comment docs.

[codecov]

Codecov Report

Merging #4211 into stable will decrease coverage by 0.14%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##           stable    #4211      +/-   ##
==========================================
- Coverage   30.28%   30.13%   -0.15%     
==========================================
  Files         209      209              
  Lines       16605    16743     +138     
  Branches     2223     2273      +50     
==========================================
+ Hits         5028     5046      +18     
- Misses      11397    11513     +116     
- Partials      180      184       +4

Impacted Files	Coverage Δ
app/app/urls.py	90% <ø> (ø)	⬆️
app/dashboard/models.py	55.74% <ø> (-0.26%)	⬇️
app/kudos/views.py	21.84% <0%> (-0.14%)	⬇️
app/event_ethdenver2019/views.py	18.6% <0%> (-0.45%)	⬇️
app/kudos/models.py	56.92% <100%> (+0.44%)	⬆️
app/dashboard/views.py	13.71% <12.5%> (-0.1%)	⬇️
app/retail/views.py	29.6% <25%> (-0.07%)	⬇️
app/grants/views.py	15.59% <0%> (+0.11%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 402af64...e961396. Read the comment docs.

[thelostone-mc]

Moving this check over to master