[13.0] Add module partner_tz

[grindtildeath]

This module removes timezone default value on res.partner and display the field
on form view.

[jgrandguillaume]

LGTM as far as I can tell

[LoisRForgeFlow]

Looks good, could you check travis error?

[LoisRForgeFlow]

@grindtildeath Hi, this module is a dependency for several modules you are proposing, if we get this merged it will ease testing and attract reviewers. Can you take a few minutes to fix travis and push this forward? 🚀

[NL66278]

Please limit the module to adding the timezone to the partner form.

But you might think about the suggested functionality of setting the default partner timezone based on the partners country...

[NL66278]

The functions presented here are very thin wrappers around the original functions from pytz. I think this is of questionable value. Better use the original pytz functions that are generally available and not depending on having this particular module installed.

[grindtildeath]

Thx for your review @NL66278
The value of a function is quite subjective in any case, but to me they make tz conversions more explicit and avoid repeats. Moreover they are only tools that won't change anything when someone installs this module.
So I'm sorry but I don't see why having this file here, and using it in other modules having this module as a dependency would be a problem...

[HaraldPanten]

Functional review

[rousseldenis]

@grindtildeath I'd prefer explicit words in module name, partner_timezone is better

[rousseldenis]

Code review. Apart module name

[rousseldenis]

And some tests for conversion functions should be great

[jgrandguillaume]

/ocabot merge nobump

[jgrandguillaume]

Merging as Alpha version is set. Thank everyone for the review.

[OCA-git-bot]

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 13.0-ocabot-merge-pr-876-by-jgrandguillaume-bump-nobump, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at 89850e4. Thanks a lot for contributing to OCA. ❤️