-
Notifications
You must be signed in to change notification settings - Fork 202
Description
Division and HrmDivision are the same model/endpoint.
Ever since we've been using the generator to keep this library up to date a duplicate (HrmDivision) got introduced.
Then there is also a SystemDivision class model/endpoint which seems to be related but then again looking at the properties seems to be different in a way. Also from my understanding we should treat them as independent models/endpoints in this library.
I think it would be best to:
- Ensure that while generating code the
HrmDivisionandSystemDivisionget updated according to their documentation - The
Divisionclass becomes aclass aliasfor theHrmDivisionor empty class extending theHrmDivisionwith a deprecation notice it will be removed in the next mayor versions
The last point might be counter intuitive as speaking about the term Division in the context of Exact Online people tend to mean the SystemDivision. But since the Division class has been the HrmDivision for over ten years switching them could potentially breaks many integrations.
Alternativly we could deprecate the HrmDivision class and make that a class alias of Division and undo the duplication that way?
Curious about what @remkobrenters and @stephangroen think about this duplication and how to move forward?