azuread_user: support employee hire date

[kang8]

Closed: #1436

[kang8]

Hi @manicminer, Can you please take a look when you have a chance?

The code has been tested on my end.

[Threpio]

Code looks good from me too (if that matters)

Just curious as to whether the time that is passed to Microsoft has to include the Hour, Minute and Second part or if it can be just a date?

[kang8]

Just curious as to whether the time that is passed to Microsoft has to include the Hour, Minute and Second part or if it can be just a date?

This is very interesting.

From the perspective of the API, it can specify hours, minutes and seconds.

But on the UI, it can only be a date.

[kang8]

Switch this PR to draft for now. After rebasing to the latest master, I noticed significant changes in the code structure. I will mark it as ready once I've finished making the necessary adjustments.

[kang8]

Hi @katbyte, I marked it as ready for this PR, and the code has been tested on my end.

Can you please take a look?

[jackofallops]

Thanks @kang8 - Just one minor change needed, then I think this will be good to go.

Thanks

[kang8]

@jackofallops thanks for your review, I've applied the patch.

[kang8]

Hi @jackofallops, could you please take another look when you have a moment? Thank you!

[jackofallops]

Thanks @kang8 for the changes, and apologies for the long delay in looping back to this, it's been a very busy few months! 🙈

This LGTM now and the tests are looking good:

[kang8]

Thanks @jackofallops for the review and merge! I was wondering, do you know when this change might be released? No rush, just curious about the schedule.