Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

NEAR testnet support #96

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

NEAR testnet support #96

wants to merge 10 commits into from

Conversation

SurgeCode
Copy link
Member

No description provided.

Copy link
Member

@bh2smith bh2smith left a comment

Choose a reason for hiding this comment

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

Seems legit. Just a lint error on CI (so far).

@@ -19,6 +19,7 @@ export interface ApiConfig {
url: string;
localAgentUrl?: string;
serverPort: number;
network: string;
Copy link
Member

Choose a reason for hiding this comment

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

Can this be an enum? "mainnet" | "testnet"?

Copy link
Contributor

@sainthiago sainthiago left a comment

Choose a reason for hiding this comment

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

code lgtm

Copy link
Member

@rubenmarcus rubenmarcus left a comment

Choose a reason for hiding this comment

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

We have one more issue

the deploy.ts script, is it deploying to a testnet registry? How user can ensure his agents are deployed on our testnet API, and also added to testnet registry?

@SurgeCode
Copy link
Member Author

We have one more issue

the deploy.ts script, is it deploying to a testnet registry? How user can ensure his agents are deployed on our testnet API, and also added to testnet registry?

That doesn't exist, its just for local testing

@bh2smith
Copy link
Member

How is everything going here with this? Are we still planning to finish here?

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

Successfully merging this pull request may close these issues.

4 participants