Skip to content

Update 03-calculate-aep-fees.mdx #2257

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

Closed
wants to merge 5 commits into from
Closed

Update 03-calculate-aep-fees.mdx #2257

wants to merge 5 commits into from

Conversation

gmehta2
Copy link
Contributor

@gmehta2 gmehta2 commented Apr 24, 2025

Copy link

vercel bot commented Apr 24, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
arbitrum-docs ✅ Ready (Inspect) Visit Preview Aug 6, 2025 2:45am

Copy link
Contributor

@TucksonDev TucksonDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @gmehta2 !
I've requested some more clarifications in some of the points 🙏

Copy link
Contributor Author

@gmehta2 gmehta2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated with feedback

  1. remove l1base fee everywhere
  2. defined some terms further

Copy link
Contributor

@TucksonDev TucksonDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Flagging some formatting errors. Also, there are 2 pending comments here:

  1. About the "daily price" (here)
  2. Mention to a "system address" (here)

@anegg0 anegg0 added the Type: Duplicate This issue or pull request already exists label Jun 4, 2025
@anegg0 anegg0 requested a review from TucksonDev August 6, 2025 02:02
@anegg0
Copy link
Contributor

anegg0 commented Aug 6, 2025

@gmehta2, The changes in this branch have already been merged to master from other AEP-related PRs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no merge conflict Type: Duplicate This issue or pull request already exists
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants