Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add configuration option for matching users with CiviCRM contacts on login #42

Merged
merged 1 commit into from
Nov 20, 2024
Merged

Add configuration option for matching users with CiviCRM contacts on login #42

merged 1 commit into from
Nov 20, 2024

Conversation

jensschuppe
Copy link
Collaborator

This adds a configuration option for matching users with CiviCRM contacts when they login:

  • when matching is generally enabled
  • only when they don't have a CiviRemote ID already
  • when they don't have any of the configured roles to exclude

systopia-reference: 22801

@jensschuppe jensschuppe added enhancement New feature or request status:needs review Code needs review and testing labels Nov 19, 2024
@jensschuppe jensschuppe added this to the 1.1.x milestone Nov 19, 2024
@jensschuppe jensschuppe requested a review from dontub November 19, 2024 14:40
dontub
dontub requested changes Nov 19, 2024
View reviewed changes
config/install/civiremote.settings.yml Show resolved Hide resolved
src/Form/CiviRemoteConfigForm.php Outdated Show resolved Hide resolved
src/Form/CiviRemoteConfigForm.php Outdated Show resolved Hide resolved
src/Form/CiviRemoteConfigForm.php Outdated Show resolved Hide resolved
config/schema/civiremote.schema.yml Show resolved Hide resolved
src/User.php Outdated Show resolved Hide resolved
@jensschuppe jensschuppe force-pushed the matchOnLogin branch 2 times, most recently from 6341029 to d48c5e0 Compare November 20, 2024 10:42
@jensschuppe jensschuppe requested a review from dontub November 20, 2024 10:43
@jensschuppe jensschuppe merged commit 9d2852c into 1.1.x Nov 20, 2024
6 checks passed
@jensschuppe jensschuppe deleted the matchOnLogin branch November 20, 2024 11:30
@jensschuppe jensschuppe added status:fixed The issue has been resolved (usually by committing/merging code) and removed status:needs review Code needs review and testing labels Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dontub dontub dontub approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request status:fixed The issue has been resolved (usually by committing/merging code)
Projects
None yet
Milestone
1.1.x
Development

Successfully merging this pull request may close these issues.
2 participants
@jensschuppe @dontub