HAL-12 open todos

[juniuszhou]

Description

Open To-dos can point to architecture or programming issues that still need to be resolved. Often these kinds of comments indicate areas of complexity or confusion for developers. This provides value and insight to an attacker who aims to cause damage to the protocol.

Code Location:

Listing 23
1 ./x/ ethbridge / test / test_helpers .go :69: // TODO : staking . TStoreKey removed in favor of? 
2 ./x/ tokenregistry / types / genesis .go :15: TODO : This is commented out because it no longer works after merging release /13.4.

Recommendation:
Consider resolving the To-dos before deploying code to a production context. Use an independent issue tracker or other project management software to track development tasks.

[juniuszhou]

too many TODOs in code, need lots time to confirm how to fix them

[pandaring2you]

Note: One 1 todo remaining for Peggy2