check for billing address before using primary address?
I think this is a feature rather than a bug, but it seems like a better way for this module to work. I've added a TODO into the code at the appropriate point in the code
add an exlude / include batched contributions into the contributions search form.
I imagine that when people have started to add lots of contributions, they'll want a way to exclude batches automatically?
should eligible for gift aid be editable once it has been submitted?
I guess it shouldn't be
Currently the code that sets the Individual declaration records when a Contribution is entered only deals with create, it ignores edit. Edit could get complicated because, particularly if admin has edited declarations in the meantime, we may not know which records to update.
Not allowing the Eligible field on the contribution to be edited sounds good to me. Guess this could be done by hiding/freezing this field when rendering the form for an edit.
If a declaration needs to be changed, an admin can edit them directly.
