Skip to content
This repository was archived by the owner on Jul 30, 2025. It is now read-only.

Conversation

@gregnazario
Copy link
Collaborator

Description

Checklist

  • If any existing pages were renamed or removed:
    • Were redirects added to next.config.mjs?
    • Did you update any relative links that pointed to the renamed / removed pages?
  • Do all Lints pass?
    • Have you ran pnpm fmt?
    • Have you ran pnpm lint?

Copilot AI review requested due to automatic review settings July 1, 2025 22:06
@gregnazario gregnazario requested a review from hariria as a code owner July 1, 2025 22:06
@vercel
Copy link

vercel bot commented Jul 1, 2025

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

Name Status Preview Comments Updated (UTC)
developer-docs-nextra ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 1, 2025 10:12pm

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds documentation and navigation entries for the new “Stateless Accounts” AIP (AIP-115).

  • Introduces an AIPs overview page with general FAQs.
  • Adds a detailed FAQ for AIP-115 (Stateless Accounts).
  • Updates meta mappings to surface the new overview and AIP-115 pages in navigation.

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
apps/nextra/pages/en/build/aips/overview.mdx Added AIPs overview page
apps/nextra/pages/en/build/aips/aip-115.mdx Added detailed FAQ for AIP-115 (Stateless Accounts)
apps/nextra/pages/en/build/aips/_meta.tsx Updated AIPs meta mapping to include overview and AIP-115
apps/nextra/pages/en/build/_meta.tsx Added “Aptos Improvement Proposals (AIPs)” to build nav
Comments suppressed due to low confidence (3)

apps/nextra/pages/en/build/aips/aip-115.mdx:29

  • [nitpick] This sentence is confusing due to the repeated use of "transaction"; consider rephrasing to something like: "When an account signs its first sequenced transaction, it..." for clarity.
When an account signs its first transaction sequence number transaction, it

apps/nextra/pages/en/build/aips/aip-115.mdx:34

  • [nitpick] The phrase "account resource" is used twice in this sentence; consider simplifying to: "For an orderless transaction, the resource is not needed and will not be created."
For an orderless transaction, the account resource is not needed at all, and the

apps/nextra/pages/en/build/aips/aip-115.mdx:120

  • [nitpick] The term "Orderless Transactions" is capitalized here but appears lowercase elsewhere (e.g., line 34); consider standardizing the capitalization throughout the doc.
### Is this compatible with Orderless Transactions?

@gregnazario gregnazario merged commit 37bfde0 into main Jul 2, 2025
4 of 5 checks passed
@gregnazario gregnazario deleted the stateless-faq branch July 2, 2025 00:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants