-
Notifications
You must be signed in to change notification settings - Fork 225
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
refactor(example): examples overhaul #309
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good! Some suggestions:
- It's prettly light on core examples so far. I'd suggest adding at least a basic protocol example here to make sure that is included in the core
- We discussed adding README.md files to the examples to briefly describe the purpose of each one
python/examples/network/01-bureau-interaction/wallet-messaging.py
Outdated
Show resolved
Hide resolved
examples overhaul
To improve maintainability (no redundancy, single source for documentation/definitions) I would suggest the following:
|
Good points @Dacksus. Definitely agree with both of the first tasks, and for these:
Are you suggesting to provide links like this for all the relevant objects / concepts: https://fetch.ai/docs/references/uagents/uagents-api/context? |
@jrriehl yes, I envision the ultimate workflow something like:
|
No description provided.