Skip to content
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.

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

Add EIP-5827: Auto-renewable allowance extension #5827

Merged
merged 32 commits into from
Nov 16, 2022

Conversation

zlace0x
Copy link
Contributor

@zlace0x zlace0x commented Oct 25, 2022

No description provided.

@zlace0x zlace0x requested a review from eth-bot as a code owner October 25, 2022 03:37
@github-actions github-actions bot added c-new Creates a brand new proposal s-draft This EIP is a Draft t-erc labels Oct 25, 2022
@eth-bot
Copy link
Collaborator

eth-bot commented Oct 25, 2022

A critical exception has occurred:
Message: pr 5827 is already merged; quitting
(cc @alita-moore, @mryalamanchi)

@github-actions github-actions bot removed c-new Creates a brand new proposal t-erc s-draft This EIP is a Draft labels Oct 25, 2022
@zlace0x zlace0x changed the title add EIP-5824: Auto-renewable allowance extension add EIP-5827: Auto-renewable allowance extension Oct 25, 2022
@github-actions github-actions bot added c-new Creates a brand new proposal s-draft This EIP is a Draft t-erc labels Oct 25, 2022
@Pandapip1 Pandapip1 changed the title add EIP-5827: Auto-renewable allowance extension Add EIP-5827: Auto-renewable allowance extension Oct 25, 2022
EIPS/eip-5827.md Outdated Show resolved Hide resolved
EIPS/eip-5827.md Outdated Show resolved Hide resolved
EIPS/eip-5827.md Outdated Show resolved Hide resolved
EIPS/eip-5827.md Outdated Show resolved Hide resolved
EIPS/eip-5827.md Outdated Show resolved Hide resolved
zlace0x and others added 2 commits October 25, 2022 22:27
Co-authored-by: Pandapip1 <45835846+Pandapip1@users.noreply.github.com>
EIPS/eip-5827.md Outdated Show resolved Hide resolved
EIPS/eip-5827.md Outdated Show resolved Hide resolved
EIPS/eip-5827.md Outdated Show resolved Hide resolved
EIPS/eip-5827.md Outdated Show resolved Hide resolved
EIPS/eip-5827.md Outdated Show resolved Hide resolved
EIPS/eip-5827.md Outdated Show resolved Hide resolved
EIPS/eip-5827.md Outdated Show resolved Hide resolved
@Rachi26
Copy link

Rachi26 commented Oct 26, 2022

#Please explain more, I don't understand

zhongfu and others added 9 commits November 3, 2022 14:35
- avoid acronyms that may be unfamiliar to audience
- reword sentences to flow better
- make them actually work
- split lines where required so max width is 100 chars
- prose fixes
- indentation
and fix typo in param name
fix: style/syntax, natspec in interfaces
…tion

fix: add Markdown link to first mention of EIP-5827
@eth-bot eth-bot enabled auto-merge (squash) November 16, 2022 14:53
@eth-bot eth-bot merged commit 3ef3772 into ethereum:master Nov 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c-new Creates a brand new proposal s-draft This EIP is a Draft t-erc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants