Skip to content

NEAR testnet support#96

Open
SurgeCode wants to merge 10 commits into
mainfrom
near-testnet
Open

NEAR testnet support#96
SurgeCode wants to merge 10 commits into
mainfrom
near-testnet

Conversation

@SurgeCode

Copy link
Copy Markdown
Member

No description provided.

@bh2smith bh2smith left a comment

Copy link
Copy Markdown
Contributor

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).

Comment thread src/commands/dev.ts
url: string;
localAgentUrl?: string;
serverPort: number;
network: string;

Copy link
Copy Markdown
Contributor

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"?

@sainthiago sainthiago left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

code lgtm

Comment thread src/playground/src/components/ChatContent.tsx Outdated
Comment thread src/playground/src/App.tsx
Comment thread src/playground/src/components/ChatContent.tsx
Comment thread src/playground/src/App.tsx

@rubenmarcus rubenmarcus left a comment

Copy link
Copy Markdown
Contributor

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
Copy Markdown
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
Copy Markdown
Contributor

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

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.

4 participants