Code review comment for lp:~kaboom/wxbanker/advancedCategories

Revision history for this message
wolfer (wolfer7) wrote :

> Hello! I just have a few initial questions.
>
> * Does this depend on lp:~kolmis/wxbanker/transaction-tagging being merged
> first?
> * It says "TODO!!! By no means
> complete!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!", is that still
> the case?

* we started our branch from the lp:~kolmis/wxbanker/transaction-tagging branch, which was not the best idea i guess. so basically our categories to not depend on transaction-tagging, but all the tagging stuff is in our branch too. i think we should either remove all the tagging stuff from our branch, or wait for the transaction-tagging branch beeing merged.

* i just removed the todo-file. there were two bugs left, but they did not reappear and i was not able to reproduce them respectively.

« Back to merge proposal