[MIG] auth_user_case_insensitive: Migration to 12.0

[ghost]

No description provided.

[pedrobaeza]

Please preserve commit history following technical method explained in https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-12.0

[ghost]

What should I do about CLA problem? I am not associated with Mr. Salmon, or LasLabs.

[pedrobaeza]

@Retropikzel don't worry about that complaint, as it's a gap in our bot.

Please fix Travis however.

[remytms]

@pedrobaeza The only error I can see in the travis check is this one:

ERROR openerp_test odoo.addons.auth_ldap.models.res_company_ldap: An LDAP exception occurred: {'desc': "Can't contact LDAP server"}

I'm not sure that this error is linked to the auth_user_case_insensitive. What do you think ?

[pedrobaeza]

Yeah, that one is the problem. You have to use mute_logger in that test for avoiding the log complaint, although you check that the raise is happening.

[remytms]

I will try to look at this tomorrow at OCA sprint.

[remytms]

Hi @pedrobaeza ,
Hi @Retropikzel ,

I have fixed the travis error with a mute_logger as suggested. See this commit coopiteasy@f67ab2f.

@Retropikzel I have created a PR to your branch https://github.com/Retropikzel/server-auth/pull/1.

[remytms]

Looks good to me. :)

[remytms]

@pedrobaeza Travis check are fixed. :) Thanks for your hints.

[pedrobaeza]

You can now look for a second reviewer reviewing you other PRs and ask for it in exchange

[jarmokortetjarvi]

Looks good, and tested the logic on runbot with different spellings of admin

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

[pedrobaeza]

Well, reviews come from non PSC members, but everything green + the hope you become more active in OCA :) get me to request the merge: /ocabot merge

[OCA-git-bot]

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 12.0-ocabot-merge-pr-104-by-pedrobaeza-bump-no, awaiting test results.

[ghost]

@pedrobaeza Thank you. :)

[OCA-git-bot]

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

[yajo]

Oops... this passed unnoticed? A commit in a test is a time bomb... 😱 💣