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 randomnet to known networks #281

Merged
merged 3 commits into from
Feb 8, 2024
Merged

Add randomnet to known networks #281

merged 3 commits into from
Feb 8, 2024

Conversation

0xaptosj
Copy link
Contributor

@0xaptosj 0xaptosj commented Feb 8, 2024

Description

Test Plan

Related Links

@0xaptosj 0xaptosj requested a review from a team as a code owner February 8, 2024 17:55
Copy link
Collaborator

@0xmaayan 0xmaayan left a comment

Choose a reason for hiding this comment

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

LGTM. Could we update CHANGELOG with this change? also, is there a fixed chain_id for randomnet?

@0xaptosj
Copy link
Contributor Author

0xaptosj commented Feb 8, 2024

LGTM. Could we update CHANGELOG with this change? also, is there a fixed chain_id for randomnet?

both addressed

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@0xmaayan
Copy link
Collaborator

0xmaayan commented Feb 8, 2024

@0xmaayan 0xmaayan self-requested a review February 8, 2024 18:26
@0xaptosj
Copy link
Contributor Author

0xaptosj commented Feb 8, 2024

sorry, could we also add tests? https://github.com/aptos-labs/aptos-ts-sdk/blob/main/tests/unit/aptosConfig.test.ts#L15

Added

@0xaptosj 0xaptosj merged commit 0b159e9 into main Feb 8, 2024
7 checks passed
@0xaptosj 0xaptosj deleted the j/add-randomnet branch February 8, 2024 18:41
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.

None yet

2 participants