Fix : disable omniauth profile edition#955
Merged
BarbaraOliveira13 merged 26 commits intodevelopfrom Apr 3, 2026
Merged
Conversation
release: v3.2.0
Release: 3.3.0
* feat: Release 3.3.0 * fix: Remove assembly update for Decidim Geo (#852)
release: 3.3.2
Release 3.4.0
Release 3.6.0
Add tracking issue templates
This PR bumps budgets_booth to its latest commit, to fix problem with the last voting popup. OpenSourcePolitics/intern-tasks#317
Release 3.9.1
* fix: udpate default partial to display custom publik translation * style: update with rubocop
* fix: Unescape HTML entities in component index checkbox tree (deface) * fix: system test
* fix: update creat_proposal extends to integrate private body from awesome * test: update creat_proposal test with private body
* fix: add validation on editor image form to avoid special characters * fix: add javascript to handle errors on editor form * test: add tests
* Revert "Update issue templates" This reverts commit 5a37ad4. * add condition * add condition in a method decidim_ai_disabled? * update default flag of decidim_ai to false * update default value and delete decidim_ai reference in text mail * debug CI, search clue * fix CI by disabled Redis for Decidim_Ai in test * initialize decidim_ai_enabled to false before secrets * fix CI * fix CI * test CI * add config analyzer in first place * clean * refacto --------- Co-authored-by: moustachu <moustachu@opensourcepolitics.eu>
* feat: update budgets_booth to latest commit * fix: add regenerator-runtime dependence to fix error
moustachu
added
Override:pending
BarbaraOliveira13
approved these changes
Mar 12, 2026
Collaborator
BarbaraOliveira13
left a comment
BarbaraOliveira13
left a comment
There was a problem hiding this comment.
LGTM 👍 Let me know If you need a third product review after deployment ...
AyakorK
approved these changes
Mar 16, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🎩 Description
This PR "fixes" (FranceConnect team point of view) the behavior of the
OMNIAUTH_FORCE_PROFILE_SYNCflag.There is a new configuration variable
OMNIAUTH_FORCE_PROFILE_SYNC_FIELDSfor selecting the fields concerned by the profile synchronization.This only works with the
nameandemail(which is the original default behavior).When
emailis not the list of synchronized field, the user has to follow the standard workflow of validating the new email address.This also fixes a SSO caveats where you need to type in your current password when changing your email :
The fixes disable the password check when there isn't any password set AND the user is currently signed in with SSO.
Testing
You need a platform with with SSO configuration and the related flag enabled.
But this has been tested by the FranceConnect qualification team on 2 platform.
🛠️ If your PR introduces or updates an Override