gnosisguild / zodiac-modifier-roles

Smart account toolkit for role-based access control
https://roles.gnosisguild.org
GNU Lesser General Public License v3.0
80 stars 39 forks source link

Improve some names in the subgraph schema #119

Closed jfschwarz closed 2 years ago

jfschwarz commented 2 years ago

Since I'm working on integrating the SDK with the subgraph and looking to use consistent naming I've stumbled over some names that I think could be improved in the subgraph schema. I'm proposing the following updates:

Parameter.theFunction -> Parameter.owningFunction Parameter.parameterIndex -> Parameter.index Parameter.parameterType -> Parameter.type Parameter.parameterComparison -> Parameter.comparison Parameter.parameterComparisonValue -> Parameter.comparisonValue

Function.functionSig -> Function.sighash

MemberRole -> RoleAssignment

RolesModifier.exec_target -> RolesModifier.target

Let me know what you think, @manboy-eth... 🙌