Merge the develop branch to the master branch, preparation to v2.0.0-rc5

This update for the master branch contains the following set of changes:
  * [Fix] Lazy setting of bridged tokens metadata (#43)
  * [Other] Another name for docker image (#38)
  * [Other] Update CI badge in README (#39)
  * [Other] Refactor of docker image to deploy and verify without flattened contracts (#40)
  * [Other] Bump package and contracts interfaces version prior to 2.0.0-rc5 (#44)

Author: akolotov

.dockerignore

@@ -12,3 +12,4 @@ flats
 contracts.sublime-project
 contracts.sublime-workspace
 build/
+package-lock.json

.eslintignore

@@ -1,3 +1,4 @@
 node_modules
 build
 coverage
+deploy/src/remix-helpers

.github/workflows/main.yml

@@ -58,7 +58,7 @@ jobs:
       - name: Prepare tag names
         id: prep
         run: |
-          DOCKER_IMAGE=poanetwork/nft-omnibridge-contracts
+          DOCKER_IMAGE=omnibridge/nft-contracts
           if [[ $GITHUB_REF == refs/tags/* ]]; then
             echo ::set-output name=tags::${DOCKER_IMAGE}:${GITHUB_REF#refs/tags/},${DOCKER_IMAGE}:latest
           else

.gitignore

@@ -8,3 +8,4 @@ deploy/*.env*
 !deploy/.env.example
 deploy/bridgeDeploymentResults.json
 coverage.json
+package-lock.json

Dockerfile

@@ -1,6 +1,6 @@
 FROM node:14 as contracts
 
-WORKDIR /contracts
+WORKDIR /workdir
 
 COPY package.json yarn.lock ./
 RUN yarn
@@ -11,14 +11,25 @@ RUN yarn compile
 
 FROM node:14
 
-WORKDIR /contracts
+WORKDIR /workdir
 
 COPY package.json yarn.lock ./
 RUN yarn install --prod
 
-COPY --from=contracts /contracts/build ./build
+# Allows remixd to be run within a docker container
+RUN sed -i s/127.0.0.1/0.0.0.0/g node_modules/@remix-project/remixd/websocket.js
+
+COPY truffle-config.js truffle-config.js
+# No need to have coverage module within the container
+RUN sed -i s/\'solidity-coverage\'\,\ // truffle-config.js
+
+COPY ./contracts ./contracts
+
+COPY --from=contracts /workdir/build ./build
 
 COPY deploy.sh deploy.sh
 COPY ./deploy ./deploy
 
-ENV PATH="/contracts/:${PATH}"
+EXPOSE 65520
+
+ENV PATH="/workdir:/workdir/node_modules/.bin:${PATH}"

README.md

@@ -1,8 +1,12 @@
 [![Join the chat at https://gitter.im/poanetwork/poa-bridge](https://badges.gitter.im/poanetwork/poa-bridge.svg)](https://gitter.im/poanetwork/poa-bridge?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)
-[![Build Status](https://github.com/poanetwork/omnibridge-nft/workflows/omnibridge-nft-contracts/badge.svg?branch=master)](https://github.com/poanetwork/omnibridge-nft/workflows/omnibridge-nft-contracts/badge.svg?branch=master)
+[![Build Status](https://github.com/omni/omnibridge-nft/actions/workflows/main.yml/badge.svg?branch=master)](https://github.com/omni/omnibridge-nft/actions/workflows/main.yml)
 
-# Omnibridge NFT Smart Contracts
-These contracts provide the core functionality for the Omnibridge NFT AMB extension.
+NFT OmniBridge Contracts
+====
+
+These contracts provide the core functionality for the NFT Omnibridge AMB extension.
+
+More details about AMB (Arbitrary Message Bridge) extensions can be found [here](https://docs.tokenbridge.net/amb-bridge/about-amb-bridge). The AMB repository is [poanetwork/tokenbridge-contracts](https://github.com/poanetwork/tokenbridge-contracts).
 
 ## License
 

REMIX.md

@@ -0,0 +1,42 @@
+How to access the contracts in the Remix IDE
+====
+
+It is possible to use the docker container to modify, test and deploy the contracts in the Remix IDE.
+
+## Read-Only Access
+
+If it is not needed to modify the contracts, they can be opened in the Remix IDE without necessity to clone the repo:
+
+1. Pull the docker image 
+   ```
+   docker pull omnibridge/nft-contracts:latest
+   ```
+
+2. Run the container
+   ```
+   docker run -d --rm --name nft-ob-remixd -p 65520:65520 
+       omnibridge/nft-contracts:latest yarn remixd
+   ```
+
+3. Open a new workspace [through connectivity to localhost](https://remix-ide.readthedocs.io/en/latest/remixd.html) in the Remix IDE. It is important to use HTTPS to access the IDE: https://remix.ethereum.org/.
+
+## Keep the changes
+
+In case of the modification of the contracts it makes sense to mount the local directory with the git repo in the Remix IDE.
+
+1. Pull the docker image 
+   ```
+   docker pull omnibridge/nft-contracts:latest
+   ```
+2. Move to the directory with the contracts.
+   ```
+   cd omnibridge-nft
+   ```
+
+3. Run the container
+   ```
+   docker run -d --rm --name nft-ob-remixd -p 65520:65520
+       -v $(pwd):/workdir omnibridge/nft-contracts:latest yarn remixd
+   ```
+
+4. Open a new workspace [through connectivity to localhost](https://remix-ide.readthedocs.io/en/latest/remixd.html) in the Remix IDE. It is important to use HTTPS to access the IDE: https://remix.ethereum.org/.

contracts/mocks/NFTWithoutMetadata.sol

@@ -0,0 +1,35 @@
+pragma solidity 0.7.5;
+
+import "@openzeppelin/contracts/token/ERC721/IERC721Receiver.sol";
+import "../interfaces/IERC1155TokenReceiver.sol";
+
+contract NFTWithoutMetadata {
+    function safeTransferFrom(
+        address _from,
+        address _to,
+        uint256 _id,
+        bytes memory _data
+    ) external {
+        IERC721Receiver to = IERC721Receiver(_to);
+        require(to.onERC721Received(msg.sender, _from, _id, _data) == to.onERC721Received.selector);
+    }
+
+    function safeTransferFrom(
+        address _from,
+        address _to,
+        uint256 _id,
+        uint256 _amount,
+        bytes memory _data
+    ) external {
+        IERC1155TokenReceiver to = IERC1155TokenReceiver(_to);
+        require(to.onERC1155Received(msg.sender, _from, _id, _amount, _data) == to.onERC1155Received.selector);
+    }
+
+    function balanceOf(address, uint256) external view returns (uint256) {
+        return 1000;
+    }
+
+    function ownerOf(uint256) external view returns (address) {
+        return msg.sender;
+    }
+}

contracts/tokens/ERC1155BridgeToken.sol

@@ -112,6 +112,18 @@ contract ERC1155BridgeToken is ERC1155, IBurnableMintableERC1155Token {
         _burnBatch(msg.sender, _tokenIds, _values);
     }
 
+    /**
+     * @dev Updated bridged token name/symbol parameters.
+     * Only bridge owner or bridge itself can call this method.
+     * @param _name new name parameter, will be saved as is, without additional suffixes like " from Mainnet".
+     * @param _symbol new symbol parameter.
+     */
+    function setMetadata(string calldata _name, string calldata _symbol) external onlyOwner {
+        require(bytes(_name).length > 0 && bytes(_symbol).length > 0);
+        name = _name;
+        symbol = _symbol;
+    }
+
     /**
      * @dev Updates the bridge contract address.
      * Can be called by bridge owner after token contract was instantiated.
@@ -170,6 +182,6 @@ contract ERC1155BridgeToken is ERC1155, IBurnableMintableERC1155Token {
             uint64 patch
         )
     {
-        return (1, 1, 0);
+        return (1, 2, 0);
     }
 }

contracts/tokens/ERC721BridgeToken.sol

@@ -77,6 +77,29 @@ contract ERC721BridgeToken is ERC721, IBurnableMintableERC721Token {
         _burn(_tokenId);
     }
 
+    // hack to access private fields in ERC721 contract
+    struct MetadataStorage {
+        string name;
+        string symbol;
+    }
+
+    /**
+     * @dev Updated bridged token name/symbol parameters.
+     * Only bridge owner or bridge itself can call this method.
+     * @param _name new name parameter, will be saved as is, without additional suffixes like " from Mainnet".
+     * @param _symbol new symbol parameter.
+     */
+    function setMetadata(string calldata _name, string calldata _symbol) external onlyOwner {
+        require(bytes(_name).length > 0 && bytes(_symbol).length > 0);
+
+        MetadataStorage storage metadata;
+        assembly {
+            metadata.slot := 6
+        }
+        metadata.name = _name;
+        metadata.symbol = _symbol;
+    }
+
     /**
      * @dev Sets the base URI for all tokens.
      * Can be called by bridge owner after token contract was instantiated.
@@ -118,6 +141,6 @@ contract ERC721BridgeToken is ERC721, IBurnableMintableERC721Token {
             uint64 patch
         )
     {
-        return (1, 0, 1);
+        return (1, 1, 0);
     }
 }