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

prepare test transactions #777

Open
wants to merge 1 commit into
base: 01-17-testing_processor
Choose a base branch
from

Conversation

yuunlimm
Copy link
Contributor

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?

Copy link

vercel bot commented Jan 17, 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 Feb 24, 2025 7:57pm

Copy link
Contributor Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@yuunlimm yuunlimm mentioned this pull request Jan 17, 2025
4 tasks
@yuunlimm yuunlimm marked this pull request as ready for review January 22, 2025 21:04
@yuunlimm yuunlimm requested a review from a team as a code owner January 22, 2025 21:04
Copy link
Collaborator

@gregnazario gregnazario left a comment

Choose a reason for hiding this comment

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

This describes it, but how do you use it?

@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from 3272d23 to 3661226 Compare January 23, 2025 18:16
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from dca6bb9 to fcc42bf Compare January 23, 2025 18:16
@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from 3661226 to 3614d53 Compare January 23, 2025 18:22
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from fcc42bf to f1853fe Compare January 23, 2025 18:22
@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from 3614d53 to a7aca8f Compare January 23, 2025 18:26
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from f1853fe to 05159b0 Compare January 23, 2025 18:26
@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from a7aca8f to b77f255 Compare January 23, 2025 18:28
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from 05159b0 to 3c2f21e Compare January 23, 2025 18:28
@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from b77f255 to e27f3c8 Compare January 23, 2025 18:30
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from 3c2f21e to 0e297bb Compare January 23, 2025 18:30
@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from e27f3c8 to f9f765c Compare January 23, 2025 18:34
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from 0e297bb to 1b3497d Compare January 23, 2025 18:34
@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from f9f765c to a287733 Compare January 23, 2025 19:47
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from 1b3497d to 6b7d434 Compare January 23, 2025 19:47
@yuunlimm yuunlimm force-pushed the 01-17-testing_processor branch from a287733 to 9992188 Compare January 23, 2025 21:17
@yuunlimm yuunlimm force-pushed the 01-17-prepare_test_transactions branch from 6b7d434 to 3c9a6ad Compare January 23, 2025 21:17
@@ -0,0 +1,18 @@
# Aptos Indexer Testing Framework Overview

The Aptos Indexer Testing Framework provides two ways to generate test transactions: **by Importing Transactions from Network** and **By writing a Move Scripts**. Both approaches are suited for specific scenarios based on your development and testing requirements, enabling you to test how your system handles various transaction types.
Copy link
Contributor

Choose a reason for hiding this comment

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

The capitalization and grammar in the two options should be consistent. Consider updating to "By Importing Transactions from Network" and "By Writing a Move Script" (note: "Move Scripts" should be singular since it refers to a single script).

Spotted by Graphite Reviewer

Is this helpful? React 👍 or 👎 to let us know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants