-
Notifications
You must be signed in to change notification settings - Fork 3
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just few small nitpicks.
A general question that I can't really assess is - is this backward compatible? Eq. for the events from the old contract will this still work?
Adam,I think it is backward compatible. But anyway we DO NOT need to be backward compatible as this will start from scratch as an new contract on the Testnet, and then on Mainnet. Future versions Might need to be Compatible for sure. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but you will have to rebase the last commit to commit-linting to pass, otherwise no merge 😅
Also if you will rename the RnsService
you will make me happy 😇
Sure will fix the commit message and the rename. |
269a7b4
to
6d935ce
Compare
6d935ce
to
1f36b7c
Compare
Kudos, SonarCloud Quality Gate passed!
|
RNS
contract tov0.1.2
RNS
simple placement contract - Update Addresses.Transfers
handler when coming form the Marketplace Contracttransfers
related to the Marketplace contract in theTokenSold
handler.registerTransfer
function to avoid code duplication as Transfers are now generated in both handlers.Works together with UI PR: rsksmart/rif-marketplace-ui#271
Setup and Deploy locally - Dev PR: rsksmart/rif-marketplace-dev#66