Skip to content

Instantly share code, notes, and snippets.

@boblail
Last active January 2, 2016 12:39
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save boblail/8304992 to your computer and use it in GitHub Desktop.
Save boblail/8304992 to your computer and use it in GitHub Desktop.
Refactor List

Ledger Refactors

  • Stop passing so many arguments to the TransactionEntry directive:
    • accounts and fiscalYears, at least, can be derived from root scope
  • Global months
  • Don't use expand and collapse methods
  • Nodes have both isAccount and resourceType
@lukebooth
Copy link

Global months

@lukebooth
Copy link

Stop passing so many arguments to the TransactionEntry directive:

  • accounts and fiscalYears, at least, can be derived from root scope

@lukebooth
Copy link

Don't use expand and collapse methods

@lukebooth
Copy link

Nodes have both isAccount and resourceType

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment