Closed (fixed)
Project:
CleverReach Module
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
13 Nov 2013 at 19:14 UTC
Updated:
27 Feb 2017 at 14:46 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
kasalla commentedNot so far. Mark this as Feature Request.
Comment #2
dercheffeI would really appreciate this feature, too.
Comment #3
kasalla commentedOk, Work is in progress,
By the end of the week a new version should be available.
Comment #4
filburt commentedHi, any news for double opt-in?
Comment #5
elbeo commentedI second this feature request.
It has been quite a long week by now ;-)
Comment #6
kasalla commentedComment #7
kasalla commentedAvailable in the new dev version.
Comment #8
derhasi commentedAs we had to revert the changes for the other tickets, this is still not implemented. I have a look on the solution discussed in IRC.
Comment #9
derhasi commentedThe attached patch implements a Double-Opt-In Feature by using the formsSendActivationMail from the CleverReach API. This means for the moment you have to select a preconfigured form from the CleverReach API (See screenshot).
This way you have a unique appearance of your confirmation mail when you use the embedded form anywhere else.
Comment #10
derhasi commentedReapplied patch after adding #2458481: Undefined index 'active' to dev.
Comment #11
derhasi commentedUpdated patch to fix form field description.
Comment #12
hctomPatch applies cleanly against the latest dev-release. I added another line in
cleverreach_subscribe_form()so thecleverreach.subscribe.incis loaded when the form is rebuilt from cache (this is for example needed when the form should be submitted via Ajax). Attached you can find the extended patch and an interdiff.Comment #13
dercheffeAny news about this issue? Unfortunately in Germany it's prohibited (data privacy laws) using a newsletter service without a double-opt-in procedure :(
Comment #14
derhasi commented@dercheffe, did you test the patch?
Comment #15
das-peter commentedI've started working on a 2.x branch (https://github.com/CandoImage/drupal_cleverreach/tree/7.x-2.x) which comes with a code skeleton for the API (Makes developing using the SOAP Service easier).
I just added this patch :) Thanks!
Comment #16
anybodyPatch #12 (or #11) does not apply cleanly against latest .dev release for me. That's curious because the patch is newer than the latest 7.x-1.x-dev release... I'll keep you updated.
Comment #17
anybodyOk I finally found out that this line leads to the problem:
; Information added by Drupal.org packaging script on 2015-03-24
version = "7.x-1.0+6-dev"
core = "7.x"
project = "cleverreach"
datestamp = "1427217788"
The patch doesn't know that line... and I think that's OK in general.
Patch #12 works great against latest .dev and I'll set it RTBC hereby. Is there an active maintainer who could create a new release containing this very important patch for German customers?
Comment #18
anybodyComment #19
braintoilet commentedYea, i will update the stable release soon to 7-1.1
Comment #21
braintoilet commentedThanks for the patch, added.
Comment #22
hoporr commentedD8 Port here: https://www.drupal.org/node/2856329