search

mikemclin / angular-acl

Role-based permissions for AngularJS
196 stars 49 forks source link

Update angular-acl.js #26

Closed tavoli closed 7 years ago

tavoli commented 7 years ago

method to drop data acl from web storage

coveralls commented 7 years ago

Coverage Status

Coverage decreased (-2.8%) to 53.913% when pulling bd6524d782384132e96c3600cad801f1e267eed4 on gustavojnt:master into 21ed26dc6d35f47d5ad31fc36f034b1c561f9781 on mikemclin:master.

mikemclin commented 7 years ago

Is there a reason while you chose the method drop? I didn't know if this was common nomenclature for removing a persisted state (like session/cache) in a different package, etc. To me, it seems more like a flush?

tavoli commented 7 years ago

The method name can be flush. Thanks for the remark!

mikemclin commented 7 years ago

Thanks for the contribution. I'll tag it in this evening.