Add support for a few additional users attributes

[michaljirman]

Add support for a few additional users attributes:

• employeeType
• costCenter
• division

Depends on Hamilton update:
PR

[manicminer]

Hi @michaljirman, thanks for this PR! Overall this looks good, I have a few suggestions inline below on the field handling in the Create/Update funcs, and also some of the ordering in the schema and docs (the linter also picked up on one of these). If we can fix these up, this should be good to merge!

[michaljirman]

Hi @michaljirman, thanks for this PR! Overall this looks good, I have a few suggestions inline below on the field handling in the Create/Update funcs, and also some of the ordering in the schema and docs (the linter also picked up on one of these). If we can fix these up, this should be good to merge!

Thanks a lot for the review! I have pushed the code changes based on the code review.
Considering this is my first contribution to the official version of this provider, please kindly review it thoroughly. It is more likely I have missed more things than it is common for the other contributors ;-)

Thanks!

[manicminer]

@michaljirman Thanks for making the changes! This mostly LGTM, I had to make a small fix upstream for the EmployeeType field which is nullable (doesn't accept blank strings). Once manicminer/hamilton#110 and then #608 are merged, this should also be good to merge 👍

[manicminer]

Test results

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.