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

feat: new almanac and name service registration flow #114

Merged
merged 8 commits into from
Jul 27, 2023

Conversation

Alejandro-Morales
Copy link
Contributor

No description provided.

@jrriehl jrriehl changed the title specify domain when registering name feat: new almanac and name service registration flow Jul 24, 2023
@jrriehl
Copy link
Contributor

jrriehl commented Jul 24, 2023

I updated this PR with a few things:

  1. Almanac registration is now performed before startup tasks. This means that actions performed on_event("startup") can now assume that registration has already taken place. This will help to prevent errors when returning messages are undeliverable.

  2. Name service registration example now uses a simple CosmPy owner wallet to illustrate the fact that domain name owners need not be and will generally not be the agents themselves.

  3. All of the contract interactions are moved to uagents.network.

Copy link
Contributor

@lrahmani lrahmani left a comment

Choose a reason for hiding this comment

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

LGTM, just a minor inline comment

examples/13-agent-name-service/agent1.py Show resolved Hide resolved
@jrriehl jrriehl merged commit b8569a5 into main Jul 27, 2023
7 checks passed
@jrriehl jrriehl deleted the domain-registration branch July 27, 2023 13:49
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

3 participants