I think there must be an option to enable/disable the syncing of accounts.
There are some use cases when we only use the modules to send emails via Mailjet.

What do you think ?

CommentFileSizeAuthor
#8 add-config-form-users-sync-3116413-7.patch4.69 KBReINFaTe
#6 add-config-form-users-sync-3116413-6.patch4.85 KBj.b
#5 add-config-form-users-sync-3116413-5.patch4.85 KBj.b
#4 add-config-form-users-sync-3116413-4.patch4.87 KBj.b
#3 3116413-add-config-form-to-activate-users-syncing.patch27.82 KBj.b

Issue fork mailjet-3116413

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Tag1 fosters the development of DrupalTag1 logo

Comments

j.b created an issue. See original summary.

j.b’s picture

j.b CreditAttribution: j.b commented
Title: Disable syncing of accounts. » Enable/disable syncing of accounts.
Issue summary: View changes
j.b’s picture

j.b CreditAttribution: j.b commented
FileSize
3116413-add-config-form-to-activate-users-syncing.patch27.82 KB

Here is a patch that adds that functionality

j.b’s picture

j.b CreditAttribution: j.b commented
FileSize
add-config-form-users-sync-3116413-4.patch4.87 KB

Patch has been rewritten

j.b’s picture

j.b CreditAttribution: j.b commented
FileSize
add-config-form-users-sync-3116413-5.patch4.85 KB
j.b’s picture

j.b CreditAttribution: j.b commented
FileSize
add-config-form-users-sync-3116413-6.patch4.85 KB
abramm’s picture

abramm
Primary language Ukrainian
Location Lutsk
CreditAttribution: abramm at DevBranch, Drupal Ukraine Community commented
Version: 8.x-2.7 » 3.0.x-dev
ReINFaTe’s picture

ReINFaTe CreditAttribution: ReINFaTe at DevBranch, Drupal Ukraine Community commented
FileSize
add-config-form-users-sync-3116413-7.patch4.69 KB

Reroll for 3.0.x

ReINFaTe opened merge request !35

ReINFaTe’s picture

ReINFaTe CreditAttribution: ReINFaTe at DevBranch, Drupal Ukraine Community commented
Status: Active » Needs review

MR !35 is hopefully a better implementation of the previous patch

SocialNicheGuru’s picture

SocialNicheGuru CreditAttribution: SocialNicheGuru commented

Does not apply to latest dev.

SocialNicheGuru’s picture

SocialNicheGuru CreditAttribution: SocialNicheGuru commented
Status: Needs review » Needs work
abramm’s picture

abramm
Primary language Ukrainian
Location Lutsk
CreditAttribution: abramm at DevBranch, Drupal Ukraine Community commented
Status: Needs work » Needs review

The MR is mergeable, so I can clearly see the changes may be applied.

SocialNicheGuru’s picture

SocialNicheGuru CreditAttribution: SocialNicheGuru commented

I can't get the MR to apply to 3.x-dev.
What am I missing?

abramm’s picture

abramm
Primary language Ukrainian
Location Lutsk
CreditAttribution: abramm at DevBranch, Drupal Ukraine Community commented
Status: Needs review » Closed (outdated)

You must be looking at wrong branch; the correct one is 3.0.x.
The latest 3.0.x-dev release track the 3.0.x branch, I've just verified it's up to date (latest file changes are 02 Mar 2023).

The 3.x-dev release was created accidentally 2 years ago and it's hidden from the releases list.

abramm’s picture

abramm
Primary language Ukrainian
Location Lutsk
CreditAttribution: abramm at DevBranch, Drupal Ukraine Community commented
Status: Closed (outdated) » Needs review
SocialNicheGuru’s picture

SocialNicheGuru CreditAttribution: SocialNicheGuru commented

You are right. I did pull down the incorrect branch via composer.
I am sorry for the noise and thank you for the solution.

mihaskep closed merge request !35