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.

Sign up for GitHub

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

Jump to bottom

StateKey -> StatefulKey #37

Closed
ro31337 opened this issue May 21, 2016 · 0 comments
Closed

StateKey -> StatefulKey #37

ro31337 opened this issue May 21, 2016 · 0 comments

Comments

@ro31337
Copy link
Owner

ro31337 commented May 21, 2016

StateKey was designed for StateStorage (Stateful now) and now should be slightly modified:

  • Rename to StatefulKey
  • guid should be optional

If guid is not specified, key should be generated the following way:

platformtype_platformid

For example:

cli_1

or

telegram_31337

StatefulKey can be used for actions (have guids), users (don't have guids), commands (may have guids).
ro31337 pushed a commit that referenced this issue May 22, 2016
#37 Rename StateKey to StatefulKey
59885cc
ro31337 pushed a commit that referenced this issue May 22, 2016
#37 Make stateful key guid optional
b2d7cd7
ro31337 added a commit that referenced this issue May 22, 2016
@ro31337
Merge pull request #40 from ro31337/37-statekey-to-statefulkey
9cd734a 
#37 StateKey to StatefulKey
@ro31337 ro31337 closed this as completed May 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ro31337