server-ux icon indicating copy to clipboard operation
server-ux copied to clipboard

[18.0][IMP] base_tier_validation: Added password confirmation

Open ArnauCForgeFlow opened this issue 2 months ago • 1 comments

Added a new field to the tier definition that allows setting a password confirmation. Later, during tier validation, the system will ask the user to confirm their password in order to validate or reject. If the comment option is enabled, the password confirmation will be requested after the comment wizard.

ArnauCForgeFlow avatar Sep 16 '25 15:09 ArnauCForgeFlow

Hi @LoisRForgeFlow, some modules you are maintaining are being modified, check this out!

OCA-git-bot avatar Sep 16 '25 15:09 OCA-git-bot

Hi @etobella ,

Could you please review this PR when you have a moment and merge it if everything looks good?

Thank you!

JoanSForgeFlow avatar Nov 29 '25 08:11 JoanSForgeFlow

@pedrobaeza could you merge here if looks good to you?

DavidJForgeFlow avatar Dec 03 '25 17:12 DavidJForgeFlow

I don't think this is correct, as you are only counting if the users have Odoo login, but not OAuth or other forms of login, and besides, it's some kind of duplicated behavior, as you already logged in. I know the login is not closed, but putting this as a way to assure impersonation due to not closing the sessions seems weird. I wouldn't definitively put this on the base module.

pedrobaeza avatar Dec 03 '25 18:12 pedrobaeza