geo/geo-coop
geo/geo-coop
7
1
Fork 0
Code Issues 55 Pull requests Projects Releases Packages Wiki Activity Actions

Restore better, built-in comment functionality to the site #12

New issue
Closed
opened 2019-07-27 19:04:08 +00:00 by mlncn · 25 comments
mlncn commented 2019-07-27 19:04:08 +00:00 (Migrated from gitlab.com)
Copy link

Per the proposal, Specific Improvements in Usability:

Fixing comments. GEO has not been satisfied with Disqus for comments as it frequently fails to load. The Drupal 8 site will handle comments itself with spam protection.

Requirements

  • Comments on all pages
  • Allow anonymous commenting, no approval required
Per the proposal, Specific Improvements in Usability: > Fixing comments. GEO has not been satisfied with Disqus for comments as it frequently fails to load. The Drupal 8 site will handle comments itself with spam protection. **Requirements** * Comments on all pages * Allow anonymous commenting, no approval required
cedewey commented 2019-08-22 18:57:24 +00:00 (Migrated from gitlab.com)
Copy link

assigned to @cedewey

assigned to @cedewey
cedewey commented 2019-08-30 22:03:05 +00:00 (Migrated from gitlab.com)
Copy link

mentioned in merge request !2

mentioned in merge request !2
cedewey commented 2019-08-30 22:03:30 +00:00 (Migrated from gitlab.com)
Copy link

assigned to @mlncn and unassigned @cedewey

assigned to @mlncn and unassigned @cedewey
cedewey commented 2019-08-30 22:03:39 +00:00 (Migrated from gitlab.com)
Copy link

Ready for your review @mlncn

Ready for your review @mlncn
cedewey commented 2019-08-31 19:01:23 +00:00 (Migrated from gitlab.com)
Copy link

changed the description

changed the description
mlncn commented 2019-09-23 00:53:36 +00:00 (Migrated from gitlab.com)
Copy link

OK first off we cannot enable anonymous commenting without any spam protection, no matter what a client says. We're the professionals here.

Adding honeypot and antibot which is what agaric.coop is using.

OK first off we cannot enable anonymous commenting without any spam protection, no matter what a client says. We're the professionals here. Adding honeypot and antibot which is what agaric.coop is using.
mlncn commented 2019-09-23 00:54:00 +00:00 (Migrated from gitlab.com)
Copy link

mentioned in commit 87cafa92a5

mentioned in commit 87cafa92a5bef7dcf202a53df2f1449d0de5fbdf
mlncn commented 2019-09-23 01:00:37 +00:00 (Migrated from gitlab.com)
Copy link

mentioned in commit cbb35dde19

mentioned in commit cbb35dde19ac86f8168a44d4f6640919c4e43676
mlncn commented 2019-09-23 01:04:31 +00:00 (Migrated from gitlab.com)
Copy link

Leaving antibot disabled for the moment... i have a soft spot for people who want to navigate the web without JavaScript (including Lynx and some screen readers, right?) We can enable if it proves necessary as an interim measure.

Leaving antibot disabled for the moment... i have a soft spot for people who want to navigate the web without JavaScript (including Lynx and some screen readers, right?) We can enable if it proves necessary as an interim measure.
mlncn commented 2019-09-23 01:06:28 +00:00 (Migrated from gitlab.com)
Copy link

mentioned in commit 35db7dd1bc

mentioned in commit 35db7dd1bc6dfb60fc0878bb386d0cd43b89fe2c
mlncn commented 2019-09-23 01:15:47 +00:00 (Migrated from gitlab.com)
Copy link

OK i just added the comment notify module so GEO members can know when there are comments and clean them up if they're spam but... that's not the module that tells admins, is it? I feel like we needed two modules to do an even mediocre version of these basic tasks...

OK i just added the comment notify module so GEO members can know when there are comments and clean them up if they're spam but... that's not the module that tells admins, is it? I feel like we needed two modules to do an even mediocre version of these basic tasks...
mlncn commented 2019-09-23 01:17:18 +00:00 (Migrated from gitlab.com)
Copy link

mentioned in commit 607be4e2b1

mentioned in commit 607be4e2b14b7741d4dff5a513d558a3967a1893
mlncn commented 2019-09-23 01:17:20 +00:00 (Migrated from gitlab.com)
Copy link

Aha yes it is "Simple Comment Notify" which i couldn't identify in our composer file because it is just scn.

Aha yes it is "Simple Comment Notify" which i couldn't identify in our composer file because it is just `scn`.
mlncn commented 2019-09-23 01:36:17 +00:00 (Migrated from gitlab.com)
Copy link

mentioned in commit c74b391332

mentioned in commit c74b39133231855de980b8e6aeb5571169be8ca3
mlncn commented 2019-09-23 01:36:17 +00:00 (Migrated from gitlab.com)
Copy link

mentioned in commit 2485830227

mentioned in commit 2485830227880cdc6408e8def304e0741ee38f67
mlncn commented 2019-09-23 01:36:17 +00:00 (Migrated from gitlab.com)
Copy link

mentioned in commit cf8c86804e

mentioned in commit cf8c86804e5005647735a1421246d3dd33a276ea
mlncn commented 2019-09-23 01:43:02 +00:00 (Migrated from gitlab.com)
Copy link
  • Simple Comment Notify is configured to send basic node/nid link of where the comment was left to site managers, people with the Manager role.
  • Comment Notify is configured to alert the people who posted the content (article, gleaning, etc) when a comment is left against it, and to give commenters the option of being notified of follow-up comments.
* Simple Comment Notify is configured to send basic node/nid link of where the comment was left to site managers, people with the Manager role. * Comment Notify is configured to alert the people who posted the content (article, gleaning, etc) when a comment is left against it, and to give commenters the option of being notified of follow-up comments.
mlncn commented 2019-09-23 01:44:52 +00:00 (Migrated from gitlab.com)
Copy link

mentioned in commit 1ebe94868b

mentioned in commit 1ebe94868bb120e7b7159b2007a164317ee6110a
mlncn commented 2019-09-23 01:46:59 +00:00 (Migrated from gitlab.com)
Copy link

assigned to @FreeScholar

assigned to @FreeScholar
mlncn commented 2019-09-23 02:19:32 +00:00 (Migrated from gitlab.com)
Copy link

mentioned in commit ba35d8c75b

mentioned in commit ba35d8c75b92ba85b4a082e7e1cb11bcde90bf18
mlncn commented 2019-10-01 02:28:04 +00:00 (Migrated from gitlab.com)
Copy link

marked this issue as related to #58

marked this issue as related to #58
mlncn commented 2019-10-01 02:28:16 +00:00 (Migrated from gitlab.com)
Copy link

marked this issue as related to #59

marked this issue as related to #59
mlncn commented 2019-10-01 02:28:38 +00:00 (Migrated from gitlab.com)
Copy link

unassigned @mlncn

unassigned @mlncn
cedewey commented 2019-10-16 20:23:21 +00:00 (Migrated from gitlab.com)
Copy link

Done!

Done!
cedewey commented 2019-10-16 20:23:21 +00:00 (Migrated from gitlab.com)
Copy link

closed

closed
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
drupal10-rebased
drupal10
master
12-comments
old-aug-left-on-local
No results found.
Labels
Clear labels   
A11y

Accessibility-related issues

  
Automated Testing

   
Contributable

Issues with work that can (potentially) be contributed back to Drupal or other FLOSS projects.

  
Contributed

Tag issues where we have contributed work back to FLOSS projects (replace Contributable tag; the Contributed tag can be removed once the contribution is recorded in a blog or something.

  
Decision

An issue that requires a decision be made.

  
Design

Issues that should include a UX/design work or review.

  
Development

   
Drutopia

Issues related to Drutopia features or infrastructure, or are potential Drutopia enhancements (with tag Contributable).

  
IndieWeb

Anything related to interlinking web sites in a broadly IndieWeb fashion, see https://indieweb.org/

  
Infrastructure

   
Launch Critical

Issue that must be done in time for launch.

  
Marketing

   
Needs documentation

When an issue has uncovered something that should be documented in a user guide or the project wiki or anywhere else outside the issue queue.

  
Post-Launch

An issue that needs to be completed, but can happen after launch.

  
status
Abandoned
An issue that has been abandoned and will not be completed.

  
status
Blocked
An issue that is blocked for a reason outside Agaric's control.

  
status
Deploy
An issue that has been tested and is ready to be deployed.

  
status
Doing

  
status
Done

  
status
In Review
An issue that has been completed and is ready for a code and/or design review.

  
status
Needs Clarification

  
status
Test
An issue that has been completed and is ready to be tested.

  
status
To Do
An issue that is an agreed priority and has all information needed to be completed.

  
type
Bug

  
type
Task

  
type
User Story

No labels
A11y
Automated Testing
Contributable
Contributed
Decision
Design
Development
Drutopia
IndieWeb
Infrastructure
Launch Critical
Marketing
Needs documentation
Post-Launch
status
Abandoned
status
Blocked
status
Deploy
status
Doing
status
Done
status
In Review
status
Needs Clarification
status
Test
status
To Do
type
Bug
type
Task
type
User Story
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: geo/geo-coop#12
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?