Version 3.5.0 (#102)

### 0.8.0
- Update sol => 0.8.0
- Update tests
- Upgrade all dependencies
- Adapt to new OnchainID version
- remove SafeMath
- add helpers + deployIdentityProxy method
- clean tests + remove redundancies
- export artifacts
- update solhint rules
- remove warning for unused variables on solidity
- import OZ contracts to contracts + update to 0.8.0
- Add SPDX Licenses
### Permissioning
- modify contracts
- add compliance manager role
- add multipurpose compliance function for custom compliance management
- add tests for all permissioning functions
- test coverage is 100%
### DvD transfers
- add DVD transfer manager contract
- add tests for DVD functions
- test coverage is 100%

includes PR #89 #90 & #96
Co-authored-by: Kevin Thizy <kevin.thizy@intech.lu>
Co-authored-by: joachimlebrun <joachim@tokeny.com>

.eslintrc.json

@@ -32,6 +32,7 @@
     "prefer-destructuring": 0,
     "prettier/prettier": "error",
     "quote-props": ["error", "as-needed"],
-    "strict": 0
+    "strict": 0,
+    "no-undef": "off"
   }
 }

.github/workflows/publish-prerelease.yml

@@ -13,8 +13,8 @@ jobs:
       - uses: actions/setup-node@v1
         with:
           node-version: 12
-      - run: yarn
-      - run: yarn build
+      - run: npm ci
+      - run: npm run build
 
   publish-npm:
     needs: build
@@ -25,9 +25,9 @@ jobs:
         with:
           node-version: 12
           registry-url: https://registry.npmjs.org/
-      - run: yarn
-      - run: yarn build
-      - run: yarn publish --tag beta
+      - run: npm ci
+      - run: npm run build
+      - run: npm publish --tag beta
         env:
           NODE_AUTH_TOKEN: ${{secrets.npm_token}}
 
@@ -41,8 +41,8 @@ jobs:
           node-version: 12
           registry-url: https://npm.pkg.github.com/
           scope: '@tokenysolutions'
-      - run: yarn
-      - run: yarn build
-      - run: yarn publish --tag beta
+      - run: npm ci
+      - run: npm run build
+      - run: npm publish --tag beta
         env:
           NODE_AUTH_TOKEN: ${{secrets.GITHUB_TOKEN}}

.github/workflows/publish-release.yml

@@ -13,8 +13,8 @@ jobs:
       - uses: actions/setup-node@v1
         with:
           node-version: 12
-      - run: yarn
-      - run: yarn test
+      - run: npm ci
+      - run: npm run build
 
   publish-npm:
     needs: build
@@ -25,9 +25,9 @@ jobs:
         with:
           node-version: 12
           registry-url: https://registry.npmjs.org/
-      - run: yarn
-      - run: yarn build
-      - run: yarn publish
+      - run: npm ci
+      - run: npm run build
+      - run: npm publish
         env:
           NODE_AUTH_TOKEN: ${{secrets.npm_token}}
 
@@ -41,8 +41,8 @@ jobs:
           node-version: 12
           registry-url: https://npm.pkg.github.com/
           scope: '@tokenysolutions'
-      - run: yarn
-      - run: yarn build
-      - run: yarn publish
+      - run: npm ci
+      - run: npm run build
+      - run: npm publish
         env:
           NODE_AUTH_TOKEN: ${{secrets.GITHUB_TOKEN}}

.github/workflows/push_checking.yml

@@ -16,20 +16,12 @@ jobs:
         uses: actions/setup-node@v1
         with:
           node-version: ${{ matrix.node-version }}
-
       - name: Install dependencies
-
-        run: yarn install --frozen-lockfile
-
+        run: npm ci
       - name: Lint sources
-        run: yarn run lint
+        run:
+          npm run lint
 
-      - name: 'Check for uncommitted changes'
-        run: |
-          git diff --exit-code --stat -- . \
-          || (echo "##[error] found changed files after build. please 'yarn install --frozen-lockfile && yarn run lint:nofix && yarn run test:unit'" \
-                   "and check in all changes" \
-              && exit 1)
   unit_test:
     name: Unit tests
     runs-on: ubuntu-latest
@@ -47,7 +39,8 @@ jobs:
           node-version: ${{ matrix.node-version }}
 
       - name: Install dependencies
-        run: yarn install --frozen-lockfile
-
+        run: npm ci
+      - name: Build application
+        run: npm run build
       - name: Run tests
-        run: yarn run test
+        run: npm run test

.gitignore

@@ -10,6 +10,5 @@ bin
 .vscode/
 
 # NPM
-package-lock.json
 gas-report
 /bin/

.solhint.json

@@ -4,13 +4,16 @@
     "func-order": "off",
     "mark-callable-contracts": "off",
     "max-states-count": "off",
-    "compiler-version": ["error", "^0.6.0"],
+    "compiler-version": ["error", "^0.8.0"],
     "no-unused-vars": ["off"],
     "no-empty-blocks": ["off"],
     "var-name-mixedcase": ["off"],
     "quotes": ["error","single"],
     "avoid-low-level-calls": ["off"],
     "no-complex-fallback": ["off"],
-    "no-inline-assembly": ["off"]
+    "no-inline-assembly": ["off"],
+    "func-name-mixedcase": ["off"],
+    "reason-string": ["off"],
+    "func-visibility": ["warn",{"ignoreConstructors":true}]
   }
 }

CHANGELOG.md

@@ -0,0 +1,37 @@
+# Change Log
+All notable changes to this project will be documented in this file.
+
+## [3.5.0]
+
+### Update
+- Updated solidity to version 0.8.0
+- Update comments in contracts code
+- Update ONCHAINID imports with the proxified version (version 1.4.0)
+- Update all dependencies to the latest stable version
+
+
+
+### Added
+- **DVDTransferManager** contract
+- Tests for DVDTransferManager functions (100% test coverage)
+- **callComplianceFunction** on OwnerManager to interact with any custom compliance function
+- complianceManager role on OwnerManager
+- exports for all contracts to be used in SDK
+- flat contract for DVDTransferManager
+- ONCHAINID proxified contract deployer for tests
+
+
+### Fix
+
+- flat contracts script : fix typo in token flattener command
+
+### Changed
+
+- Release & pre release flows (yarn -> npm)
+- removed useless lines on `removeClaimTopics` function
+- add try/catch in `isVerified` function to avoid errors with incompatible claims
+- removed useless lines on `removeTrustedIssuer` function
+- changed required key for roles on AgentManager contract (MANAGEMENT key -> EXECUTION key)
+- changed required key for roles on OwnerManager contract (MANAGEMENT key -> EXECUTION key)
+- import interfaces from openzeppelin instead of local copy
+- contract name : **Storage** -> **TokenStorage**