# Summary

Comment Notify sends notification e-mails to visitors about new comments on pages where they have commented.

Initial port committed, with a few essential followups identified.

# Project URL

https://www.drupal.org/project/comment_notify

# Where is the code?

#2221061: Port comment_notify to Drupal 8

# Estimated completion date

January 31, 2016

# Dependencies

# Who's doing the port?

Arla at MD Systems

# What help do they need?

Feel free to start/join on any of the followup issues on refactoring and test coverage.

# D8 roadmap

None

# Background and reference information

None

Comments

Arla created an issue. See original summary.

arla’s picture

arla commented
Issue summary: View changes
jonathanshaw’s picture

jonathanshaw
Primary language English
Location Stroud, UK
commented
Status: Needs work » Needs review

Module seems to be described as usable

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented
Status: Needs review » Needs work

Until there is an alpha or a beta, this should be at 'needs work'.

mmjvb’s picture

mmjvb commented
Status: Needs work » Needs review
galania’s picture

galania commented

i try install, but i have this error :

Uncaught PHP Exception Drupal\\Core\\Database\\IntegrityConstraintViolationException: "SQLSTATE[23502]: Not null violation: 7 ERROR: el valor null para la columna \xc2\xabnotify_hash\xc2\xbb viola la restricci\xc3\xb3n not null\nDETAIL: La fila que falla contiene (3, 0, null, 0).: INSERT INTO comment_notify (cid, notify, notify_hash) SELECT c.cid AS cid, 0 AS notify, MD5(:salt || c.mail || COALESCE(u.mail, u.init) || c.uid || c.name || c.entity_id || c.hostname || c.cid) AS notify_hash\nFROM \ncomment_field_data c\nINNER JOIN users_field_data u ON c.uid = u.uid; Array\n(\n)\n" at /home/web/core/lib/Drupal/Core/Database/Connection.php line 683

gnuget’s picture

gnuget
Primary language Spanish
commented
Status: Needs review » Fixed

We just released the first stable version of the module, so I'm going to mark this as fixed. :-)

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Assigned: arla » Unassigned

I am giving credits to the users who participated in this issue.

Status: Fixed » Closed (fixed)

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