Show subscription date/time in Drupal Admin [#2222151]

Comment	File	Size	Author
#3	simplenews-Show_subscription_date_time_in_Drupal_Admin-2222151-2-7.x-2.x.patch	1.94 KB	sandip27
7.x-2.x: PHP 5.4 & MySQL 5.5, D7 Skipped, branch did not pass for 4 weeks
#2	edit_show_subscription-2222151-1.patch	1.76 KB	rcodina
7.x-1.x: PHP 5.4 & MySQL 5.5, D7 46 pass, 7 fail PHP 5.3 & MySQL 5.5, D7 47 pass, 6 fail PHP 7.2 & MySQL 5.5, D7 46 pass, 10 fail

Comment #1

rcodina CreditAttribution: rcodina commented 20 July 2016 at 10:13

Related issues:

+#948200: Timestamp for subscribers

I add related issue for D6.

Log in or register to post comments

File	Size
edit_show_subscription-2222151-1.patch	1.76 KB
7.x-1.x: PHP 5.4 & MySQL 5.5, D7 46 pass, 7 fail PHP 5.3 & MySQL 5.5, D7 47 pass, 6 fail PHP 7.2 & MySQL 5.5, D7 46 pass, 10 fail

Comment #3

sandip27 CreditAttribution: sandip27 at Cybage Software Pvt Ltd. commented 20 July 2016 at 10:55

File	Size
simplenews-Show_subscription_date_time_in_Drupal_Admin-2222151-2-7.x-2.x.patch	1.94 KB
7.x-2.x: PHP 5.4 & MySQL 5.5, D7 Skipped, branch did not pass for 4 weeks

@miro_dietiker, Please find attached patch for your review. Would be great if you could merge the same.

Thaks

Log in or register to post comments

Comment #4

sandip27 CreditAttribution: sandip27 at Cybage Software Pvt Ltd. commented 20 July 2016 at 11:23

Assigned:

Unassigned

» sandip27

Log in or register to post comments

Comment #5

sandip27 CreditAttribution: sandip27 at Cybage Software Pvt Ltd. commented 16 August 2016 at 06:53

Assigned:

sandip27

» Unassigned

Log in or register to post comments

Comment #6

rcodina CreditAttribution: rcodina commented 16 August 2016 at 07:33

@sandip27 Please attach and interdiff with my patch

EDIT: Nevermind, your patch is against another branch

Log in or register to post comments

Comment #7

sandip27 CreditAttribution: sandip27 at Cybage Software Pvt Ltd. commented 15 December 2016 at 14:39

Priority:

Normal

» Critical

Log in or register to post comments

Comment #8

Neo13 CreditAttribution: Neo13 commented 26 June 2017 at 09:57

This looks solid. Please maintainer check against latest dev and commit.

Log in or register to post comments

Comment #9

rcodina CreditAttribution: rcodina commented 26 June 2017 at 18:56

Status:

Needs review

» Reviewed & tested by the community

Log in or register to post comments

Comment #10

sandip27 CreditAttribution: sandip27 commented 3 February 2018 at 04:16

Hi, patch provided is reviewed and tested by @rcodina so, any word on applying this patch ?

Log in or register to post comments

Comment #11

mgstables

Dutch

CreditAttribution: mgstables commented 13 May 2018 at 17:21

Hi, i did try the patch from #2 and it did nothing. I dont see a new table column with "creation date". Then i tested patch from #3, also no result.
Maybe i do something wrong? It would be nice to have a creation date, especially now with the new GDPR law. (sorry for bad english)
simplenews 7.x-1.x-dev
Drupal 7.59

update: Sorry, i just find out that the simplenews_realname module was the problem. So it works, i just had to alter also the simplenews_realname module.