Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Commit

Permalink
Backport #6710 & #6714 (#6715)
Browse files Browse the repository at this point in the history
* [ci skip] js-precompiled 20171011-183908

* Fix estimate gas if from is not provided. (#6714)

* Display vouched overlay on dapps (#6710)

* Remove .only

* Add vouch overlays to dapps

* Cleanup address

* Only run where we have a contentHash

* GitLab kickstart
  • Loading branch information
jacogr authored and arkpar committed Oct 11, 2017
1 parent d30c715 commit 6d0dfa6
Show file tree
Hide file tree
Showing 11 changed files with 204 additions and 8 deletions.
2 changes: 1 addition & 1 deletion js/scripts/test.js
Original file line number Diff line number Diff line change
@@ -1 +1 @@
// test script 10
// test script 11
1 change: 1 addition & 0 deletions js/src/contracts/abi/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,3 +29,4 @@ export signaturereg from './signaturereg.json';
export smsverification from './sms-verification.json';
export tokenreg from './tokenreg.json';
export foundationWallet from './foundation-multisig-wallet.json';
export vouchfor from './vouchfor.json';
1 change: 1 addition & 0 deletions js/src/contracts/abi/vouchfor.json
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
[{"constant":true,"inputs":[],"name":"certifier","outputs":[{"name":"","type":"address"}],"payable":false,"stateMutability":"view","type":"function"},{"constant":false,"inputs":[{"name":"_what","type":"bytes32"}],"name":"vouch","outputs":[],"payable":false,"stateMutability":"nonpayable","type":"function"},{"constant":true,"inputs":[{"name":"_what","type":"bytes32"},{"name":"_index","type":"uint256"}],"name":"vouched","outputs":[{"name":"","type":"address"}],"payable":false,"stateMutability":"view","type":"function"},{"constant":false,"inputs":[{"name":"_what","type":"bytes32"},{"name":"_index","type":"uint256"}],"name":"unvouch","outputs":[],"payable":false,"stateMutability":"nonpayable","type":"function"},{"constant":true,"inputs":[{"name":"","type":"bytes32"},{"name":"","type":"uint256"}],"name":"vouchers","outputs":[{"name":"","type":"address"}],"payable":false,"stateMutability":"view","type":"function"},{"inputs":[{"name":"_certifier","type":"address"}],"payable":false,"stateMutability":"nonpayable","type":"constructor"},{"anonymous":false,"inputs":[{"indexed":false,"name":"who","type":"address"},{"indexed":false,"name":"what","type":"bytes32"}],"name":"Vouched","type":"event"}]
2 changes: 2 additions & 0 deletions js/src/ui/DappCard/dappCard.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ import React, { Component, PropTypes } from 'react';
import Container, { Title as ContainerTitle } from '~/ui/Container';
import DappIcon from '~/ui/DappIcon';
import Tags from '~/ui/Tags';
import DappVouchFor from '../DappVouchFor';

import styles from './dappCard.css';

Expand Down Expand Up @@ -61,6 +62,7 @@ export default class DappCard extends Component {
app={ app }
className={ styles.image }
/>
<DappVouchFor app={ app } />
<Tags
className={ styles.tags }
tags={
Expand Down
43 changes: 43 additions & 0 deletions js/src/ui/DappVouchFor/dappVouchFor.css
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
/* Copyright 2015-2017 Parity Technologies (UK) Ltd.
/* This file is part of Parity.
/*
/* Parity is free software: you can redistribute it and/or modify
/* it under the terms of the GNU General Public License as published by
/* the Free Software Foundation, either version 3 of the License, or
/* (at your option) any later version.
/*
/* Parity is distributed in the hope that it will be useful,
/* but WITHOUT ANY WARRANTY; without even the implied warranty of
/* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
/* GNU General Public License for more details.
/*
/* You should have received a copy of the GNU General Public License
/* along with Parity. If not, see <http://www.gnu.org/licenses/>.
*/

.tag {
color: inherit;
position: absolute;
top: 1em;
right: 1em;

.image {
position: absolute;
right: 0;
top: 0;
width: 32px;
height: 32px;
}

.bubble {
background: red;
border-radius: 0.25em;
color: white;
font-size: 0.75em;
padding: 0.1em 0.5em;
position: absolute;
right: 0;
top: 0;
z-index: 100;
}
}
57 changes: 57 additions & 0 deletions js/src/ui/DappVouchFor/dappVouchFor.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,57 @@
// Copyright 2015-2017 Parity Technologies (UK) Ltd.
// This file is part of Parity.

// Parity is free software: you can redistribute it and/or modify
// it under the terms of the GNU General Public License as published by
// the Free Software Foundation, either version 3 of the License, or
// (at your option) any later version.

// Parity is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
// GNU General Public License for more details.

// You should have received a copy of the GNU General Public License
// along with Parity. If not, see <http://www.gnu.org/licenses/>.

import React, { Component, PropTypes } from 'react';
import { observer } from 'mobx-react';

import IdentityIcon from '../IdentityIcon';

import Store from './store';
import styles from './dappVouchFor.css';

@observer
export default class DappVouchFor extends Component {
static contextTypes = {
api: PropTypes.object.isRequired
};

static propTypes = {
app: PropTypes.object.isRequired
};

store = new Store(this.context.api, this.props.app);

render () {
const count = this.store.vouchers.length;

if (!count) {
return null;
}

return (
<div className={ styles.tag }>
<IdentityIcon
address={ this.store.vouchers[0] }
className={ styles.image }
alt={ `${count} identities vouch for this dapp` }
/>
<div className={ styles.bubble }>
{ count }
</div>
</div>
);
}
}
17 changes: 17 additions & 0 deletions js/src/ui/DappVouchFor/index.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
// Copyright 2015-2017 Parity Technologies (UK) Ltd.
// This file is part of Parity.

// Parity is free software: you can redistribute it and/or modify
// it under the terms of the GNU General Public License as published by
// the Free Software Foundation, either version 3 of the License, or
// (at your option) any later version.

// Parity is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
// GNU General Public License for more details.

// You should have received a copy of the GNU General Public License
// along with Parity. If not, see <http://www.gnu.org/licenses/>.

export default from './dappVouchFor';
74 changes: 74 additions & 0 deletions js/src/ui/DappVouchFor/store.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,74 @@
// Copyright 2015-2017 Parity Technologies (UK) Ltd.
// This file is part of Parity.

// Parity is free software: you can redistribute it and/or modify
// it under the terms of the GNU General Public License as published by
// the Free Software Foundation, either version 3 of the License, or
// (at your option) any later version.

// Parity is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
// GNU General Public License for more details.

// You should have received a copy of the GNU General Public License
// along with Parity. If not, see <http://www.gnu.org/licenses/>.

import { action, observable } from 'mobx';
import { uniq } from 'lodash';

import Contracts from '~/contracts';
import { vouchfor as vouchForAbi } from '~/contracts/abi';

export default class Store {
@observable vouchers = [];

constructor (api, app) {
this._api = api;

const { contentHash } = app;

if (contentHash) {
this.lookupVouchers(contentHash);
}
}

lookupVouchers (contentHash) {
Contracts
.get().registry
.lookupAddress('vouchfor')
.then((address) => {
if (!address || /^0x0*$/.test(address)) {
return;
}

return this._api.newContract(vouchForAbi, address);
})
.then(async (contract) => {
if (!contract) {
return;
}

let lastItem = false;

for (let index = 0; !lastItem; index++) {
const voucher = await contract.instance.vouched.call({}, [`0x${contentHash}`, index]);

if (/^0x0*$/.test(voucher)) {
lastItem = true;
} else {
this.addVoucher(voucher);
}
}
})
.catch((error) => {
console.error('vouchFor', error);

return;
});
}

@action addVoucher = (voucher) => {
this.vouchers = uniq([].concat(this.vouchers.peek(), [voucher]));
}
}
4 changes: 3 additions & 1 deletion js/src/ui/IdentityIcon/identityIcon.js
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ class IdentityIcon extends Component {

static propTypes = {
address: PropTypes.string,
alt: PropTypes.string,
button: PropTypes.bool,
center: PropTypes.bool,
className: PropTypes.string,
Expand Down Expand Up @@ -84,7 +85,7 @@ class IdentityIcon extends Component {
}

render () {
const { address, button, className, center, disabled, inline, padded, tiny } = this.props;
const { address, alt, button, className, center, disabled, inline, padded, tiny } = this.props;
const { iconsrc } = this.state;
const classes = [
styles.icon,
Expand Down Expand Up @@ -135,6 +136,7 @@ class IdentityIcon extends Component {

return (
<img
alt={ alt || address }
className={ classes }
data-address-img
height={ size }
Expand Down
9 changes: 4 additions & 5 deletions js/src/util/tx.js
Original file line number Diff line number Diff line change
Expand Up @@ -81,12 +81,11 @@ export function getTxOptions (api, func, _options, values = []) {
options.to = options.to || func.contract.address;
}

if (!address) {
return Promise.resolve({ func, options, values });
}
const promise = (!address)
? Promise.resolve(false)
: WalletsUtils.isWallet(api, address);

return WalletsUtils
.isWallet(api, address)
return promise
.then((isWallet) => {
if (!isWallet) {
return { func, options, values };
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ describe('views/Signer/components/SignRequest', () => {
expect(isMarkdown(encodedMd)).to.be.true;
});

it('returns false for randow data', () => {
it('returns false for random data', () => {
expect(isMarkdown('0x1234')).to.be.false;
});
});
Expand Down

0 comments on commit 6d0dfa6

Please sign in to comment.