Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove sender in AddHash / removeHash events #10

Open
sponnet opened this issue Apr 12, 2018 · 0 comments
Open

remove sender in AddHash / removeHash events #10

sponnet opened this issue Apr 12, 2018 · 0 comments
Assignees

Comments

@sponnet
Copy link
Member

sponnet commented Apr 12, 2018

The member ( where the quota needs to be kept track of ) of the pinning request can be figured out from the transaction itself ( the sender of the transaction is either a member , or the contract address that emmitted the event is a member of the consortium )

This logic can all be inspected from the proxy.

Skipping this saves gas cost on the event.

@sponnet sponnet self-assigned this Apr 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant