EIP Editing Office Hour Meeting 26

[poojaranjan]

Date & Time

Tuesday, October 31, 2023, at 15:00 UTC

Location

Zoom: TBA on EthCatHerders Discord #eip-editors
Recording: https://www.youtube.com/playlist?list=PL4cwHXAawZxqnDHxOyuwMpyt5s8F8gdmO

Agenda

• Visit EIPs
• Rename it to ERC Editing Office Hour?
• Next meeting time?

To Final

• Update ERC-7066: Move to Last Call ethereum/ERCs#47

To Last Call

• Update ERC-3643: Move to Last Call ethereum/ERCs#44
• Update ERC-6982: Move to Last Call ethereum/ERCs#29

To Review

• Update ERC-6956: Move to Review ethereum/ERCs#43
• Update ERC-7092: Move to Review ethereum/ERCs#19
• Update ERC-6596: Move to Review ethereum/ERCs#38
• Update ERC-6956: Move to Review ethereum/ERCs#43
• Update ERC-7540: Move to Review ethereum/ERCs#52
• Update ERC-7535: Move to Review ethereum/ERCs#78

To Withdrawan

TBA

Misc PR

• Update ERC-6358: Add Compatibility with the Metamask(EIP-712) ethereum/ERCs#40
• Fixed links in ERCS markdown files ethereum/ERCs#42
• Update ERC-5564: missing necessary arg in computeStealthKey ethereum/ERCs#72

To Draft

Requested in comments

• Add ERC: Content Consent for AI/ML Data Mining ethereum/ERCs#80
• Add ERC: NFT Dynamic Traits ethereum/ERCs#26
• Add ERC: NFT Redeemables ethereum/ERCs#27
• Add ERC: Trusted Hint Registry ethereum/ERCs#21

Added from the past meetings
TBA

From Satgnant

TBA

Old links

• Update EIP-5521: Move to Review ethereum/EIPs#7632
• Update EIP-7092: Move to Review ethereum/EIPs#7667
• Add EIP: Entity Component System ethereum/EIPs#7651
• Add EIP: Dynamic Compliant Interop Security Token ethereum/EIPs#7719
• Add EIP: Content Consent for AI/ML Data Mining ethereum/EIPs#7682
• Add EIP: Trusted Hint Registry ethereum/EIPs#7547
• Add EIP: NFT Dynamic Traits ethereum/EIPs#7500
• Add EIP: NFT Redeemables ethereum/EIPs#7501
• Add EIP: Registries and Adapters for Smart Accounts ethereum/EIPs#7484
• Update EIP-7053: Move to Final ethereum/EIPs#7804

Open Q&A

Other details

• Announcement Blogpost

(Add your Pull Request number to be added to the agenda.)

[OniReimu]

Hi @poojaranjan , could you please include ethereum/EIPs#7632, thanks.

[HelloRickey]

Hi @poojaranjan please take a look at
ethereum/EIPs#7651
thanks

[d4mr]

Hi @poojaranjan can ethereum/EIPs#7719 be included as well please,
thanks

[bafu]

Hi Pooja, I would like to discuss ethereum/EIPs#7682 (EIP-7517) and move it to Draft. Thank you!

[bafu]

Hi Pooja, I would like to discuss ethereum/EIPs#7804 (EIP-7053) and move it to Final. Thank you!

[poojaranjan]

In the absence of the EIP editor, we're skipping this week. Hopefully, PRs will be merged async in the meantime. If not, it will be reviewed at the next meeting.

[Edoumou]

Hi Pooja, I would like to discuss ethereum/EIPs#7667 (ERC-7092) and move it to Review. Thank you.

[kopy-kat]

Hi Pooja, I would like to discuss ethereum/EIPs#7484 and merge the first draft. Thank you!

[strumswell]

Review of ethereum/EIPs#7547 would be appreciated.

[poojaranjan]

In the absence of the EIP editor, we're skipping this week. Hopefully, PRs will be merged async in the meantime. If not, it will be reviewed at the next meeting on October 31, 2023.

[ryanio]

Hi Pooja 👋 I would like to discuss adding these EIPs (sync or async is fine!):
Add EIP: NFT Dynamic Traits #7500
Add EIP: NFT Redeemables #7501 (uses Dynamic Traits)

Have had an initial review/approval from @g11tech but need ✅ from @SamWilsn since he is still in "Requested Changes" state for the EIP numbers update (which has been done).

[poojaranjan]

@ryanio
I have added it to the agenda. However, I see the requested changes by two different reviewers. I recommend reaching both of them on EthCatHerders Discord #eip-editors channel.

[tbergmueller]

Hi @poojaranjan ,

we'd like to move ERC-6956 to review and discuss any open poins in the upcoming meeting, in case not approved before.
ethereum/EIPs#7903

Side-note, small typo in your ticket description, From Satgnant ;)

[dhl]

Hi @poojaranjan,

My team have requested for ERC-6596 to be
moved to review state, and would like to bring this up for discussion during the office hour if the changes have not been merged by then. Would you mind including this on the agenda please?

[Edoumou]

Hello @poojaranjan,

I was wondering if one should reference the new PR after moving ERCs to a separated directory.
If so, then I would like to discuss ethereum/ERCs#19 (ERC-7092) and move it to Review. Thank you.

[strumswell]

Same for me ethereum/ERCs#21

[tbergmueller]

Same for me;
Instead of ethereum/EIPs#7903
please use
ethereum/ERCs#43

Note that for all PRs at the moment pipelines in the ERCs repo fail, mostly due to spelling mistakes and broken links in other ERCs. These are known issues and being actively worked on I believe

[Joeysantoro]

Can we add: ethereum/ERCs#78 to move ERC-7535 to Review and ethereum/ERCs#57 to fix a typo on final ERC-2612?

[poojaranjan]

@Joeysantoro ethereum/ERCs#78 has been added to this meeting. However ERC-2612 is in Final status, it is added to the EIPIP meeting.

[bafu]

Hi Pooja, I would like to discuss ethereum/EIPs#7682 (EIP-7517) and move it to Draft. Thank you!

[dhl]

Hi @poojaranjan! The invite link to discord doesn't work. Would you mind sharing the link?

[poojaranjan]

@dhl ECH Discord

[poojaranjan]

Pull requests that couldn't be discussed today are added to the next meeting. But we hope for them to be merged before the next EIP Editing Office Hour.

[poojaranjan]

Closed in favor of #290