Migrated to OZ v5 #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

Arvolear merged 40 commits into master from dev

Apr 22, 2025

Contributor

KyrylR commented Mar 24, 2025 •

edited by aritkulova

Loading

Since this PR suggests a bug fix, the tests have been added and the coverage is 100%.
Since this PR introduces a new feature, the update has been discussed in an Issue or with the team.
This PR is just a minor change, like a typo fix.


          Migrated to OZ v5

8ad57a3

KyrylR marked this pull request as draft

March 24, 2025 12:43

aritkulova added 11 commits

March 26, 2025 16:49


          integrated with new versions of solarity and openzeppelin

bed7a53


          updated package-lock

9812e15


          switched to custom errors

c382599


          cleaned up

23ace8e


          refactored error naming

1118e7f


          renamed claimTopicKey to contextKey, claimTopic to handleTopic

764dc9a


          fixed typos

c9fe3e0


          fixed inits in example

b73bba0


          returned accidentally missing part

58c7181


          updated examples folder

aac0246


          Added erc721 (#7)

8d8d53d

* created transaction context for both nft and token

* added NFT F initial version

* updated naming

* updated dependencies

* fixed typo

* updated readme

* added example of NFTF contract

* review fixes

* handle topic -> handler topic

* updated package-lock files

Arvolear reviewed

View reviewed changes

contracts/core/Globals.sol Outdated Show resolved Hide resolved

Arvolear reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved

Arvolear reviewed

View reviewed changes

contracts/core/Globals.sol Outdated Show resolved Hide resolved

Arvolear reviewed

View reviewed changes

contracts/modules/AbstractModule.sol Outdated Show resolved Hide resolved

Arvolear reviewed

View reviewed changes

contracts/modules/AbstractModule.sol Outdated Show resolved Hide resolved

Arvolear reviewed

View reviewed changes

contracts/modules/AbstractModule.sol Outdated Show resolved Hide resolved

Arvolear reviewed

View reviewed changes

examples/equity-token/contracts/EquityKYCCompliance.sol Outdated Show resolved Hide resolved

Arvolear reviewed

View reviewed changes

examples/equity-token/contracts/EquityRegulatoryCompliance.sol Outdated Show resolved Hide resolved

Arvolear reviewed

View reviewed changes

hardhat.config.ts Outdated Show resolved Hide resolved

Arvolear reviewed

View reviewed changes

package.json Outdated Show resolved Hide resolved

Arvolear reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved

aritkulova added 6 commits

April 8, 2025 18:43


          fixed typos

554b25e


          created IAssetF interface for Context and common errors

cbfa65d


          added onlyInitializing to init functions

29e3462


          updated minimal solidity version to ^0.8.21

54539df


          added storage buckets to the modules

e3fb73f


          updated the architecture diagram

4f42e86

aritkulova added 2 commits

April 14, 2025 15:18


          updated dependency

383263f


          fixed diagram

c7aaa95

Arvolear reviewed

View reviewed changes

contracts/core/NFTF.sol Outdated Show resolved Hide resolved

Arvolear and others added 4 commits

April 16, 2025 13:44


          few fixes

3bccb9f


          moved TokenF and NFTF contracts from core folder

bede6ef


          updated diagram

ce475a0


          calldata -> memory in public functions

3442b57

Arvolear approved these changes

View reviewed changes

Member

Arvolear left a comment

LFG! (need to update example after merge)

aritkulova added 2 commits

April 16, 2025 21:21


          TransferLimitsModule -> ERC20TransferLimitsModule

04e3b12


          EquityNFT -> LandNFT

4829b56

Hrom131 reviewed

View reviewed changes

contracts/NFTF.sol Outdated Show resolved Hide resolved

Member

Hrom131 commented Apr 17, 2025

Let's create AbstractAssetF contract for the TokenF and NFTF contracts

aritkulova added 6 commits

April 17, 2025 18:10


          added ERC721TransferLimitsModule

f3d61ec


          added LandERC721TransferLimitsModule to examples

dfc5a46


          added INFTF to supported interfaces

af81d51


          updated scripts

d6b676a


          added AbstractAssetF contract

8ecf012


          order consistency fix

798f81e

Arvolear reviewed

View reviewed changes

contracts/modules/regulatory/ERC721TransferLimitsModule.sol Outdated Show resolved Hide resolved

aritkulova added 2 commits

April 18, 2025 17:06


          fixed ERC721TransferLimitsModule formula

8d39297


          cleaned up ERC721TransferLimitsModule test

9bddc87

Arvolear reviewed

View reviewed changes

contracts/modules/regulatory/ERC721TransferLimitsModule.sol Outdated Show resolved Hide resolved

Arvolear reviewed

View reviewed changes

contracts/modules/regulatory/ERC721TransferLimitsModule.sol Outdated Show resolved Hide resolved

aritkulova added 2 commits

April 20, 2025 16:05


          review fix

6b1dead


          updated diagram

80fa17b

Arvolear reviewed

View reviewed changes

test/modules/regulatory/ERC721TransferLimitsModule.test.ts Outdated Show resolved Hide resolved

fix

Arvolear approved these changes

View reviewed changes

Member

Arvolear left a comment

LGTM!

Arvolear merged commit 9aaabd4 into master

1 check passed

Arvolear deleted the dev branch

April 22, 2025 10:52

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet