Implement Ownership and Access Control Facets

[mudgen]

See @andrew-fleming's comment about access control here: #8 (comment)

We will need an OwnershipFacet that implements ERC-173: Contract Ownership Standard.

We should also implement a facet for role based access/permissions.

[0xlynett]

#17

[Rushikesh0125]

gm @mudgen, would like to pick this up!

I like the idea of Andrew about access control methods being intuitively readable and relate to similar usage.

[mudgen]

@Rushikesh0125 Excellent. @0xlynett already implemented OwnershipTransfer functionality, but you can implement the roles based access functionality. Do you want to do that?

[0xlynett]

whoops am also already doing RBAC lol

[mudgen]

@Rushikesh0125, @0xlynett maybe you guys can work together on this somehow. What do you think?

[Rushikesh0125]

@0xlynett @mudgen Sorry for this clash and bad comms

#20 I opened a PR here, would request you guys to check it out

[0xlynett]

Oh sorry lol I'll check out your PR and then work on the enumerable variant then

[adamgall]

this issue can be closed, because #20 was merged (cc: @mudgen)

[mudgen]

Great