FINERACT-2425: Tax component and Tax Group refactoring

[alberto-art3ch]

Description

The Taxes module is one of the oldest functionality, and It has some issues, like:

• typo issues,
• the data readers are implemented with old logic

Then it is not useful then the APIs, write and read, are not working

FINERACT-2425

Checklist

Please make sure these boxes are checked before submitting your pull request - thanks!

• Write the commit message as per our guidelines
• Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.
• Create/update unit or integration tests for verifying the changes made.
• Follow our coding conventions.
• Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/legacy-docs/apiLive.htm with details of any API changes
• This PR must not be a "code dump". Large changes can be made in a branch, with assistance. Ask for help on the developer mailing list.

Your assigned reviewer(s) will follow our guidelines for code reviews.

[Aman-Mittal]

Code wise seems ok, I did not reviewed it based on backward compatibility point of view

[bharathcgowda]

@adamsaghy could you please help in reviewing this PR when you get some time?

[adamsaghy]

LGTM