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

Frank update #1

Merged
merged 5 commits into from
Mar 20, 2025
Merged

Frank update #1

merged 5 commits into from
Mar 20, 2025

Conversation

QingyangKong
Copy link
Collaborator

remove the comments
update prompt and description
update readme

@QingyangKong QingyangKong requested review from pappas999 and zeuslawyer and removed request for pappas999 March 11, 2025 04:20
zeuslawyer
zeuslawyer previously approved these changes Mar 12, 2025
zeuslawyer
zeuslawyer previously approved these changes Mar 19, 2025
Copy link
Contributor

@zeuslawyer zeuslawyer left a comment

Choose a reason for hiding this comment

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

LGTM - please note the comments. Thanks!

README.md Outdated

Open `src/character.ts` to modify the default character. Uncomment and edit.
The agent uses custom actions that interact with [Chainlink Functions](https://docs.chain.link/chainlink-functions).
Copy link
Contributor

Choose a reason for hiding this comment

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

Perhaps hyperlink to the actions docs in ElizaOS docs so that people know what we mean by "Actions"

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Good point. Updated

It will give you a menu of all the packages to approve. You can just type `a` and it will select all. Then type `Y` as per the prompt.

Note: this requires node to be at least version 22 when you install packages and run the agent.
> The full gitbook walkthrough and tutorial is available at : https://cll-devrel.gitbook.io/elizaos-functions-twitter/
Copy link
Contributor

Choose a reason for hiding this comment

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

any updates needed to gitbook?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, I will update the url in gitbook too.

Copy link
Contributor

@zeuslawyer zeuslawyer left a comment

Choose a reason for hiding this comment

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

LGTM

@QingyangKong QingyangKong merged commit 3833214 into main Mar 20, 2025
1 check passed
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.

2 participants