Problem/Motivation

Within Dblog or maillog there are no reports after a user has logged in and email sent the code, is there a way to add reporting in?

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork email_tfa-3441944

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

Comments

ben.campbell created an issue. See original summary.

aaron.ferris made their first commit to this issue’s fork.

aaron.ferris opened merge request !13

aaron.ferris’s picture

aaron.ferris commented

Raised an MR for 2.x, ill need to look at 1.0.7 (Sorry didn't realise this was raised against 1.x).

aaron.ferris opened merge request !14

aaron.ferris changed the visibility of the branch 3441944-log-emails-sent to hidden.

aaron.ferris’s picture

aaron.ferris commented
Status: Active » Needs review
rollins’s picture

rollins commented
Status: Needs review » Reviewed & tested by the community

Changes from MR look good to me, it makes sense to have logs after a user has logged in and email sent the code
I think this can be merged

ben.campbell’s picture

ben.campbell commented

This is fantastic thank you so much, tested the MR and works great

abdulaziz zaid’s picture

abdulaziz zaid
Primary language Arabic
Location Riyadh 🇸🇦
commented

Thanks for this work, but I suggest setting the Feature on settings for admin to enable log or disable log.

abdulaziz zaid’s picture

abdulaziz zaid
Primary language Arabic
Location Riyadh 🇸🇦
commented
Status: Reviewed & tested by the community » Needs work
aaron.ferris’s picture

aaron.ferris commented

Ah yes, good idea, ill look at that.

aaron.ferris’s picture

aaron.ferris commented
Assigned: Unassigned » aaron.ferris
aaron.ferris’s picture

aaron.ferris commented
Status: Needs work » Needs review
aaron.ferris’s picture

aaron.ferris commented

Added admin option to both MRs.

abdulaziz zaid’s picture

abdulaziz zaid
Primary language Arabic
Location Riyadh 🇸🇦
commented

Merged ! 13 an 14 🎉

abdulaziz zaid’s picture

abdulaziz zaid
Primary language Arabic
Location Riyadh 🇸🇦
commented
Status: Needs review » Fixed

Thank you all

abdulaziz zaid’s picture

abdulaziz zaid
Primary language Arabic
Location Riyadh 🇸🇦
commented
Version: 1.0.7 » 2.0.x-dev

Status: Fixed » Closed (fixed)

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