Add support for tokens BNTY and MKR about requestnetwork-v1-archive HOT 10 CLOSED

HI Guys,

I have prepared the library to support the two tokens: #4

The next steps to close the issue are:

1. Change the burner address @tipsysquid
2. Transfer the ownership @tipsysquid
3. Update the fees on the contracts @vrolland
4. Autorized the contracts on the request core @vrolland
5. Test creation and payments @vrolland
6. Merge & Publish Library and Artifacts changes @vrolland
7. Update the doc @vrolland
8. Have a beer 🍺 (all - optional)

Cheers,

from requestnetwork-v1-archive.

Hey guys any update on this?

from requestnetwork-v1-archive.

Hi, thanks for creating this issue. We haven't had time to make progress on this, we will do it end of this week or beginning of the next.
I had a quick look and noticed that the burner address points to the old one, the correct one is 0x7b3c4d90e8af6030d66c07f8f815f9505e379d6f. Do you remember where you took it from? There may be information to update somewhere

from requestnetwork-v1-archive.

from requestnetwork-v1-archive.

Hi @romaric-juniet ,
I may have made this issue against the incorrect repo.
I was using documentation for v1 and using v1 resources; however, I did not make this issue for v1.
Here is where I found the burner address that I used.

I'm happy to redeploy the contracts with the correct burner address.

Should I close this issue and re-open it under requestNetwork-v1-archive?

from requestnetwork-v1-archive.

@romaric-juniet any update on this?

Thanks,

from requestnetwork-v1-archive.

Hi Guys,

I am taking over the issue.

@tipsysquid
Thanks for the work done!

I was using the documentation for v1 and using v1 resources; however, I did not make this issue for v1. Here is where I found the burner address that I used.

It was confusing. I will try to make it more clear and clean the deploy file which is out of date.

I'm happy to redeploy the contracts with the correct burner address.

You don't need to redeploy the contract to change the burner address. You can use the function setRequestBurnerContract()
You will need to give us at some point the ownership of the contracts. You can wait for me to check everything first or go head using transferOwnership() with the request admin address 0x74ef019c1e9f11366c5c8dc4ab556c16fe13b51f

Should I close this issue and re-open it under requestNetwork-v1-archive?

Yep, It would be a good idea actually.

@AngeloAdam
I will start to check everything tomorrow. (SGT time)

Cheers,

from requestnetwork-v1-archive.

I have completed the following:

• Change the burner addresses

• Transfer ownership to Request Admin address 0x74ef019c1e9f11366c5c8dc4ab556c16fe13b51f

You can follow the transactions from the previous owner address here

Thanks for providing the guidelines @vrolland! 🚀

from requestnetwork-v1-archive.

@AngeloAdam Can you contact me in private on the slack "request hub" please?

from requestnetwork-v1-archive.

done #7 🍻

from requestnetwork-v1-archive.