wharfkit / session

Create account-based sessions, perform transactions, and allow users to login using Antelope-based blockchains.
Other
9 stars 1 forks source link

Rename permissionLevel on the transaction context #17

Closed aaroncox closed 1 year ago

aaroncox commented 1 year ago

The term permissionLevel may be more complicated than needed to explain what this property is. The value of this property is actually a PermissionLevel object containing something like foobar@active, but could be either split or some other variation.

It could potentially be named something related to "Account", but the TransactContext.account will be an instance of the Account Kit. Unsure what to rename it to yet.

aaroncox commented 1 year ago

Perhaps allow specifying either permissionLevel or actor and permission?

This change can be made on both the TransactContext for plugin developers and the Session constructor to allow application developers an alternative.

The permissionLevel property on these objects can remain and be a PermissionLevel core object for those using it.