jaheyns
Reporting Function for Y+
I am happy to do the work to add a new reporting function for Y+.
I notice that the current code, and especially the UI file for reporting functions, is already becoming somewhat mixed with the three functions that are supported. In the interest of making it easier to add and maintain more functions in the future. Would you rather I create it in its own files or just continue the pattern of adding it to the existing reporting functionality?
I would also be happy to do a bit of a refactor with the current functionality, but I assume that may not be backwards compatible?
I am happy to do the work to add a new reporting function for Y+.
That would be great, thanks! :)
I notice that the current code, and especially the UI file for reporting functions, is already becoming somewhat mixed with the three functions that are supported. In the interest of making it easier to add and maintain more functions in the future. Would you rather I create it in its own files or just continue the pattern of adding it to the existing reporting functionality?
A separate file would be better, I think. The original implementation was a bit haphazard if I remember correctly.
I would also be happy to do a bit of a refactor with the current functionality, but I assume that may not be backwards compatible?
If you feel like restructuring a bit, please go ahead - I think it would be worth taking the hit on backward compatibility for better organisation here.
