Problem/Motivation

Taking part of the MR from #3312998: Explain how to use tokens immediately without going to confirmation page .

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#6 simplenews.immediate-confirm.3390913-interdiff-2-6.txt1.75 KBadamps
#6 simplenews.immediate-confirm.3390913-6.patch1.84 KBadamps
#3 simplenews.immediate-confirm.3390913-2.patch1.66 KBadamps

Comments

AdamPS created an issue. See original summary.

AdamPS credited szeidler.

adamps’s picture

adamps commented
StatusFileSize
new simplenews.immediate-confirm.3390913-2.patch1.66 KB
adamps’s picture

adamps commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 3: simplenews.immediate-confirm.3390913-2.patch, failed testing. View results

adamps’s picture

adamps commented
Status: Needs work » Needs review
StatusFileSize
new simplenews.immediate-confirm.3390913-6.patch1.84 KB
new simplenews.immediate-confirm.3390913-interdiff-2-6.txt1.75 KB
szeidler’s picture

szeidler
Primary language German
Location Berlin
commented
Status: Needs review » Reviewed & tested by the community

Thanks for splitting up the issue and providing the patch. It fixes the issue.

  • AdamPS committed 157b5a2a on 4.x 
    Issue #3390913 by AdamPS, szeidler: Immediate confirmation is broken in...
adamps’s picture

adamps commented
Status: Reviewed & tested by the community » Fixed

Thanks

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.