Do you mean "do we need the review" or do you mean "do we need percentage-based transfer fees"?
AFAICS the latter has already been decided by the shareholders, and regarding the former I would say that yes, a review is needed before we merge the code.
I thought about fees a little too ...
do you thing we can have something like this:
The asset issue sets an attribute for his asset on "what fee type is used". Then we can offer
* flat fee (BTS only) .. no fee pool
* flat fee with fee pool
* percentage based fee (BTS only)
* percentage based fee with fee pool
* bandwidth-based according to BTS reserved by issuer
* bandwidth-based according to BTS reserved by user (only for committee-owned/approved assets)
* any other type of fee that can/could/may be implemented
How difficult would it be to modify the code such that the "fee" part of it becomes more "plugin"-ish?