Skip to content

[16.0][FIX] password_security: Fix expected singleton#762

Merged
OCA-git-bot merged 1 commit into
OCA:16.0from
vauxoo-dev:16.0-fix-singleton-moy
Mar 11, 2025
Merged

[16.0][FIX] password_security: Fix expected singleton#762
OCA-git-bot merged 1 commit into
OCA:16.0from
vauxoo-dev:16.0-fix-singleton-moy

Conversation

@moylop260

@moylop260 moylop260 commented Feb 12, 2025

Copy link
Copy Markdown
Contributor

Updating the password for more than one user raises expected singleton error

This fix supports multi case

Comment thread password_security/models/res_users.py Outdated
Comment thread password_security/models/res_users.py Outdated
@luisg123v

Copy link
Copy Markdown
Contributor

Commit message:

"Updating the password for more than one user it raises": remove "it"

@moylop260 moylop260 force-pushed the 16.0-fix-singleton-moy branch 2 times, most recently from c172e8d to 1152250 Compare February 13, 2025 02:57
@StefanRijnhart StefanRijnhart changed the title [FIX] password_security: Fix expected singleton [16.0][FIX] password_security: Fix expected singleton Mar 3, 2025
@pedrobaeza pedrobaeza added this to the 16.0 milestone Mar 3, 2025

@pedrobaeza pedrobaeza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-762-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Mar 3, 2025
Signed-off-by pedrobaeza
@OCA-git-bot

Copy link
Copy Markdown
Contributor

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-762-by-pedrobaeza-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

Updating the password for more than one user raises "expected singleton" error

This fix supports multi case
@antonag32 antonag32 force-pushed the 16.0-fix-singleton-moy branch from 1152250 to b990a54 Compare March 11, 2025 01:15
@antonag32

Copy link
Copy Markdown

Hello @pedrobaeza, can you issue the merge command again? I rebased the PR so that it uses the updated version of workflow files, it should not fail anymore.

@pedrobaeza

Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-762-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit c5fe554 into OCA:16.0 Mar 11, 2025
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@moylop260 moylop260 deleted the 16.0-fix-singleton-moy branch March 13, 2025 21:20
moylop260 added a commit to vauxoo-dev/server-auth that referenced this pull request Jun 24, 2026
Updating the password for more than one user raises "expected singleton" error

This fix supports multi case

(cherry picked from commit OCA@b990a54 OCA#762)
@moylop260

Copy link
Copy Markdown
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants