Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

TokenReg unregister should only be visible for dapp contract owner #3321

Closed
jacogr opened this issue Nov 9, 2016 · 0 comments
Closed

TokenReg unregister should only be visible for dapp contract owner #3321

jacogr opened this issue Nov 9, 2016 · 0 comments
Assignees
@ngotchac
Labels
F2-bug 🐞 The client fails to follow expected behavior. M8-dapp 💎 Decentralized applications.

Comments

@jacogr
Copy link
Contributor

jacogr commented Nov 9, 2016

As per implementation, unregister is available for contract owner, not token owner (currently we have it swapped around)
@jacogr jacogr added F2-bug 🐞 The client fails to follow expected behavior. M8-dapp 💎 Decentralized applications. labels Nov 9, 2016
@ngotchac ngotchac self-assigned this Nov 10, 2016
ngotchac added a commit that referenced this issue Nov 10, 2016
@ngotchac
Fixed Unregister for Contract Owner only #3321
58d3e99
@ngotchac ngotchac mentioned this issue Nov 10, 2016
Merged
gavofyork pushed a commit that referenced this issue Nov 10, 2016
@ngotchac @gavofyork
Fixed Unregister for Contract Owner only #3321 (#3346)
cb84e61
@jacogr jacogr mentioned this issue Nov 11, 2016
Closed
arkpar pushed a commit that referenced this issue Nov 15, 2016
@ngotchac @arkpar
Fixed Unregister for Contract Owner only #3321 (#3346)
c171207
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ngotchac ngotchac

Labels
F2-bug 🐞 The client fails to follow expected behavior. M8-dapp 💎 Decentralized applications.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jacogr @ngotchac