mirror of
				https://github.com/0glabs/0g-chain.git
				synced 2025-11-04 01:37:28 +00:00 
			
		
		
		
	prioritise todos
This commit is contained in:
		
							parent
							
								
									bbe83c6ad7
								
							
						
					
					
						commit
						3ae217e927
					
				@ -7,16 +7,17 @@ Simplifications:
 | 
				
			|||||||
 | 
					
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 TODO
 | 
					 TODO
 | 
				
			||||||
 | 
					 - in code TODOs
 | 
				
			||||||
 | 
					 - write basic cmds
 | 
				
			||||||
 | 
					 - Tidy up - method descriptions, heading comments, remove uneccessary comments, README/docs
 | 
				
			||||||
 - chnge module name to "channel"?
 | 
					 - chnge module name to "channel"?
 | 
				
			||||||
 - Find a better name for Queue - clarify distinction between int slice and abstract queue concept
 | 
					 - Find a better name for Queue - clarify distinction between int slice and abstract queue concept
 | 
				
			||||||
 - refactor queue into one object
 | 
					 - write some sort of integration test
 | 
				
			||||||
 - Do all the small functions need to be methods on the keeper or can they just be floating around?
 | 
					 | 
				
			||||||
 - Tidy up - standardise var names, method descriptions, heading comments
 | 
					 | 
				
			||||||
 - any problem in signing your own address?
 | 
					 | 
				
			||||||
 - Gas
 | 
					 | 
				
			||||||
 - find nicer name for payout
 | 
					 - find nicer name for payout
 | 
				
			||||||
 - tags - return channel id
 | 
					 - add Gas usage
 | 
				
			||||||
 - create custom errors instead of using sdk.ErrInternal
 | 
					 - add tags (return channel id on creation)
 | 
				
			||||||
 - maybe split off signatures from update as with txs/msgs - testing easier, code easier to use, doesn't store sigs unecessarily on chain
 | 
					 - use custom errors instead of using sdk.ErrInternal
 | 
				
			||||||
 | 
					 - split off signatures from update as with txs/msgs - testing easier, code easier to use, doesn't store sigs unecessarily on chain
 | 
				
			||||||
 - consider removing pubKey from UpdateSignature - instead let channel module access accountMapper
 | 
					 - consider removing pubKey from UpdateSignature - instead let channel module access accountMapper
 | 
				
			||||||
 - remove printout from tests when app initialised
 | 
					 - remove printout from tests when app initialised
 | 
				
			||||||
 | 
					 - refactor queue into one object
 | 
				
			||||||
 | 
				
			|||||||
		Loading…
	
		Reference in New Issue
	
	Block a user