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

Figure out actor state #164

Closed
SeanMcOwen opened this issue Sep 25, 2023 · 2 comments · Fixed by #180
Closed

Figure out actor state #164

SeanMcOwen opened this issue Sep 25, 2023 · 2 comments · Fixed by #180
Labels
help wanted Extra attention is needed

Comments

@SeanMcOwen
Copy link
Collaborator

Probably want to see what will be tech debt and removed versus not with pocket team

message Actor {
ActorType actor_type = 1;
string address = 2;
string public_key = 3;
repeated string chains = 4; // NB: Not applicable Validator actors
// proto-gen-c does not support go_name at the time of writing resulting
// in the output go field being snakeCase: ServiceUrl (golang/protobuf#555)
string service_url = 5; // Not applicable to Application actors
string staked_amount = 6;
int64 paused_height = 7; // TECHDEBT: Revisit this parameter and see if it can be removed for simplification purposes.
int64 unstaking_height = 8; // TECHDEBT: Revisit this parameter and see if it can be removed for simplification purposes.
string output = 9; // TECHDEBT: Revisit custodial / non-custodial flows (e.g. what if we want multiple outputs for business purposes)
}

@SeanMcOwen
Copy link
Collaborator Author

Do we want to use this is a base class for other instantiated entities?

@SeanMcOwen SeanMcOwen added the help wanted Extra attention is needed label Sep 27, 2023
@SeanMcOwen SeanMcOwen linked a pull request Sep 29, 2023 that will close this issue
@SeanMcOwen
Copy link
Collaborator Author

Last aspects represented with latest branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant