benk10

I'm pretty sure that the transition Vyper had in wei_value from ``int128`` to ``uint256`` was after that. I'll go through the code and write the comments.

I like the implementation, looks really interesting and useful. I agree with @leviadam an EIP for this can be good as it's very generalized and I think it can be...

Regarding the first comment, I think we should implement [EIP165](https://github.com/ethereum/EIPs/blob/master/EIPS/eip-165.md) which specifies how to do that. I'm currently trying to solve the issue you mentioned, will update when I'm finished.

@dev-matan-tsuberi I am still trying to figure this out, currently my thought here is to use an array on Keys instead of a single key so you might have a...

@dev-matan-tsuberi I get multiple compile errors on the original code. Some small things but there's a problem with passing a dynamic sized array as the modifier parameter. It can be...

@dev-matan-tsuberi But you will still have problem with the transferable permission. What I meant is that instead of having one Key per id we can have array of Keys. Each...

This should be working now but I'm unable to test it as I don't have Windows, closing for now and please reopen if this is still relevant.

1. Using anything which has balances (reputation, tokens, etc.), just making Infra more generalized. 2. The event was emitting `reputation` variable which was the user's total reputation instead of `rep`...

This is currently blocked due to a Graph Node bug. I will follow up with them on that and return to it when possible.

https://github.com/graphprotocol/graph-cli/pull/477