EIP Editing Office Hour #27
EIP Editing Office Hours · 14 Nov 2023 · 52:30
Meeting 27 of the EIP Editing Office Hour, held on 14 November 2023. EIP editors and proposal authors work through open pull requests and questions about the Ethereum standards process. This meeting covers ERC pull requests heading to Final, Last Call, Review and Draft along with miscellaneous pull requests.
Topics
From EIP Editing Office Hours, hosted by ECH Institute.
Transcript
welcome to EIP editing office hour meeting 27 thank you Sam and authors for joining us we as we can see we are joined by multiple authors uh so we would like to take the PO request uh for them specifically to begin with so we can answer their questions and then we will take rest of the PLL request added to the agenda so we may start with ERC PR number 77 It Is by team combo I guess one combo yeah combo all right let me just share my screen here and then we can uh go into
that uh okay there so uh did you have any specific questions yeah we don't have have other specific questions all right I take a look at it then all right open IP protocol remix nfts and N Works while their relationships can be T chain Okay
so oh okay okay uh got that good all that looks good right this protoc a us how to remix existing IPS and create a new IP so when you say IP here you probably mean intellectual property yes yes
so okay it extends the your c721 standard enabling users to remix multiple existing nfts and create a new nft derivative work while the relationships can be traced on the blockchain all right so um the abstract it should okay so the inter is flood with fresh
content every day but with traditional IP infrastructure IP registration and Licensing is a headache for digital creators um okay the rer creation of content has eclips a slower pace of Ip registration leaving much of this content unprotected this means creators can't fairly earn from their Works spread okay uh traditional IP infrastructure open IP infrastructures IP registration okay backrop or passion about building open IP okay sure
um this prot how to remix multiple existing nfts and create a new nft derivative work known as a combo while their relationships can be traced on the blockchain it contains three core modules remix module network module and license module all right so the remix module extends erc721 standard 72155 okay license module okay the one premise two extensions one cost
principal one premise means that for any IP in the open source ecosystem and nft stands for that IP so if you have the nft you own the IP that's for the open IP protocol okay two extensions okay okay so um for
okay okay compatibility the H 720 [Music] yep okay cool yeah so I think this is good
um feel free to give me a ping once you make the changes and uh yeah should be able to merge that cool thank you Sam no no problem all right yeah please go ahead Sam oh where we have to next so uh next we are joined by two authors for proposal in last call let's start with ERC pr29 and it is Francesco is here yeso hey did you have any uh specific questions about this uh not really we are using production in many projects and U uh now is going for a while there have been a lot of discussion about little
details to improve but the community agree on where we are now that is is a simple and um and covers the the case that we are trying to solve the basically is to avoid using wasting gas when a soulbound token or a badge is is is created because the other the alternative right now is to emit a locked event or an unlocked event any times but if you know if it's if it's a badge I know that all the tokens are locked forever so that is why there is this
default loc event it solves the problem of the yep consumption yeah perfect um so you've checked your erc1 165 interface ID uh yes perfect okay okay good yeah I don't think this has changed much since the last time I looked at it right oh yeah yeah it's we didn't CH we just made some little change in the document to fix some type or some improved a
little bit the the English there was some mistake and that is it so no that's perfect yeah well this's merge itself or am I going to have to merge this I'll leave that one open where we off to next okay and the next one is 47 47 all right we are joined by push to talk about the proposal if there is any
question hello s yeah there's no specific question just uh all the comments that you had I think I've resolved those and just wanted to get your update on this for moving to final yeah okay um
see may I ask a question yeah of course it's I saw that in this in the in the guide we should keep the link document like e721 here line 58 y but here has been changed to erc721 so which is the the update strategy should we move to ERC or we keep EIP EIP would be the correct one okay um yeah as
far as I'm aware um Matt and Panda pip have done a lot more work on the split than I have so I'm actually not sure what's correct um but I think if the guide says use EIP then you should use EIP oh okay thank you I was on assumption that the previous ones would would get renamed so I just put ERC for the new one but if it's to uh I wouldn't worry like too too much about it we'll fix them like if we have to so yeah I guess the better question here would be Sam will it uh the EIP W bot
will allow both EIP and ERC at this point of time so it it only allows ERC right now and I think that's incorrect so yeah okay yeah okay so that one merged Perfect all right um so the next one we have is for review status and the pull request number is 19 19 all right okay
okay Okay cool so let's take a look uh this proposal introduces fixed income fential bonds and this was going to review this changes was proposed by light clients I think okay yeah yeah so and there there
are some changes that is proposed by you I think like uh moving back the commands uh into the the standard and also removing the optional in the interface in functions putting uh maybe uh saying that the world interface is optional but not particular functions was done yep yeah I saw that um perfect and the transfer all this function is remove because because uh for security purposes okay so prevent users to manage all the balance if they want to do it then they need to uh input
the W amount in yeah okay good okay good yeah I think this looks this looks good I think uh you made all the changes I asked for and yeah so
perfect um so I'll just approve this um I just noticed typo feel free to fix it whenever you want um yeah awesome easy all right so the next one is uh again a review proposal but uh the author isn't present and he has left his comments for you Sam to follow and the questions that he has uh this one here yes yeah
so right yeah okay for all right so that was
19 right um now I think we can take it sequentially so we have one proposal in panel we can start from there and go okay
good and then let's go make sure that there's no further discussion on this I
okay yeah otherwise this looks good okay so I'll keep that one open so Sam um in the meantime we are joined by another author and the pull request number is 52 52 all right okay all right you see 4626 F
standard SM composable Atomic deposits this liation do not work well for any smart contract with asynchronous AC sear Lanes so we are joined by Jo for this proposal in case there is any question hey Joey he Z okay um not
implemented okay the synch deposit B must override the near C 4626 specification as follows the deposit and mint methods do not transfer asset to the Vault because happened on Quest deposit 46 okay submission request go through
pending claim claim stages example life cycle for deposit request request okay I apologize for the ammo oh it's fine
um okay yeah this looks perfect definitely ready to go to review so yeah so just fix that at your leisure um yeah otherwise looks great Sam I have a question yeah uh so for the proposal like uh the the P request for which the comment was left it is for a
PR 43 um do we want to maybe take a look if that PR is ready to be merged oh sure yeah okay so this is going from draft to review asset non tokens ass B Anor token to to an asset operations are authorized through Oracle attestation over control over the asset all right this standard allows integrating physical and digital assets without signing capabilities into DBS we 3 by extending your c721 asset a physical object Mark with unique the
identifiable anchor the anchor oh I remember this one okay the three testation theal testifies to that particular asset is associated with an anchor it's been [Music] controlled okay okay the wellknown [Music] 21 okay elev the content
change of current okay 57
91 okay standard okay subject C below okay mechanism okay okay wa what is this block of code
minimal typescript sample to generate an attestation using ether's Library um so okay
good okay so that's an interface that's fine okay good those are good rationals excellent use cases possession based use case covered by the standard interface the holder okay the leaves lending
okay and I think everything else looks
good yeah okay cool thank you so we can go back to the last call I think the pr number is 96 96 all right um moving into last call so see if there's been any discussion on here no
discussion perfect okay the standard extends EC 721 by bring individual identities to nonp tokens and sbound tokens by buing multiple identities okay okay one of the most interesting aspects x 3 is the ability to bring an individuals own identity to different applications even more powerful is the fact that individuals truly own their accounts without relying on centralized Gatekeepers to skills into different apps components necessary for authentication approv by individuals decentralized datation okay so you have a
interface neat metadata schema perfect following okay mated okay some test cases reference implementation yeah I mean that seems good so the only thing to change is the last call deadline date yeah so that'll be
11 28 cool on to 44 quick question Sam um the on 7540 it there's like the HTML proofer is
failing so I can't merge the it shouldn't be a required check um cuz I mean it might be um HTML proofer isn't required so you should be able to merge it okay for some reason I'm not able to maybe is it the auto review trigger bot that's the only one which uh which poll request number was it 52 52 oh yeah that's the required one that's failing that first one hey look at that that's super useful I know
um so we will do this and then we'll do this and we will watch it can I ask another question while we wait yeah of course it's I have another proposal that I'm I'm doing and it's we are still discussing the proposal so it's not urgent but all the errors that I have are related with other uh proposals uh not with
the I can link here the the link to the report if you want take a look put in the chat you can see that all the errors are related with the um with yes in the HTML profiler you can see it I Rel with other documents not to document the time yep yeah it checks the whole repo so you just have to ignore it um it won't stop you from merging okay so it's fine okay yeah yeah okay thank you oh no problem yeah so what
is going on with this one a little bit better this time I think let's see here nope it failed again okay I will bring this up with um Matt and Panda and I'll see what's going on sorry about that yeah no worries thanks for looking at that so I guess we're just like in holding patterns now for that one yeah I'll uh yeah I'll get back to you cool yeah no worries thanks okay let's get this one a
read okay actually what were my other comments on this and was it just the date was just the date but I think otherwise this was pretty good
okay yeah this is fine and then we just have to adjust the last call deadline so we will do that after are you're going to merge come
on these Bots are taking their oh yeah yeah they really are there we go so this is 3643 so we go over here okay fix that
problem all right now wait I need that one still okay so on to 34 multiple met you the option okay this one propos an extension the RP 721 standard to support multiple metadata U per token introdu a new interface compatible okay
4906 okay C5 B C5 okay this seems good we have a reference implementation nice so the only thing
I need I mean 13 days I think is close enough we're going to go for that right yeah a okay so then we have 6596 okay cultural and historical asset toen meditated to enhance the discoverability connectivity and collec collectability of culturally and historically significant NTS okay
good that looks good this C propos the establishment of comprehensive standard for cultural and hisorical asset tokens on The Ether platform these tokens represent cultural and historical assets such as artwork artifacts Collectibles rare providing crucial Contex and provin okay cultural historical importance value of such asset for wi but adoption okay T in the blockchain okay good so there's a
footnote where's your footnote what to find for
for okay search discovery that okay exist most oper on trust block teolog off okay good good okay
um sh history of
artifacts distributed comprehensive contact and substance
found okay meditator interface okay for
okay no I mean that's probably fine you just have to add new ones and new eips so it's okay both the ERC 72 1155 provide natural extension points good okay attributes property good backward compatibility okay yeah it seems fine all right cool and with that that is all the time I
have for today so I will see you guys in two weeks thank you sir in the meantime if you get a chance please maybe take a look at miscellaneous PR if there is anything that needs to be move to the EIP meeting you may leave a comment and I will add that yeah thank you everyone yeah thank you very much