Remove uage of history module from comment module

Created on 14 May 2025, 2 months ago

Problem/Motivation

History module is being deprecated.

Steps to reproduce

Proposed resolution

TBD

Remaining tasks

TBD

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

📌 Task
Status

Active

Version

11.0 🔥

Component

history.module

Created by

🇳🇿New Zealand quietone

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @quietone
  • 🇦🇺Australia mstrelan

    We might need to split this up further, this is what I've identified.

    PHP:

    • \Drupal\comment\CommentLinkBuilder::buildCommentedEntityLinks
    • \Drupal\comment\CommentManager::getCountNewComments
    • \Drupal\comment\CommentManagerInterface::getCountNewComments
    • \Drupal\comment\CommentViewBuilder::buildComponents
    • \Drupal\comment\Hook\CommentHooks::nodeViewAlter
    • \Drupal\comment\Hook\CommentViewsHooks::viewsDataAlter
    • \Drupal\comment\Plugin\views\field\NodeNewComments::preRender
    • \Drupal\Tests\comment\Functional\CommentCSSTest::testCommentClasses
    • \Drupal\Tests\comment\Functional\CommentEntityTest::testEntityChanges
    • \Drupal\Tests\comment\Functional\CommentNewIndicatorTest::testCommentNewCommentsIndicator
    • \Drupal\Tests\comment\Functional\CommentTestBase::$modules
    • \Drupal\Tests\comment\Functional\Views\NodeCommentsTest::$modules
    • \Drupal\Tests\comment\Unit\CommentLinkBuilderTest::testCommentLinkBuilder

    Libraries:

    • drupal.comment-new-indicator
    • drupal.node-new-comments-link

    Views:

    • test_new_comments
  • Pipeline finished with Failed
    6 days ago
    Total: 251s
    #552676
  • 🇦🇺Australia mstrelan
  • 🇦🇺Australia mstrelan
  • Pipeline finished with Failed
    6 days ago
    Total: 185s
    #552681
  • Pipeline finished with Failed
    6 days ago
    Total: 180s
    #552700
  • Pipeline finished with Failed
    6 days ago
    Total: 530s
    #552706
  • Pipeline finished with Canceled
    6 days ago
    Total: 193s
    #552725
  • Pipeline finished with Success
    6 days ago
    Total: 614s
    #552726
  • 🇦🇺Australia mstrelan
  • Pipeline finished with Failed
    6 days ago
    Total: 199s
    #552739
  • Pipeline finished with Failed
    6 days ago
    Total: 652s
    #552741
  • Pipeline finished with Running
    5 days ago
    #553496
  • Pipeline finished with Failed
    5 days ago
    #553499
  • Pipeline finished with Success
    5 days ago
    #553502
  • Pipeline finished with Running
    5 days ago
    #553533
  • 🇦🇺Australia mstrelan
Production build 0.71.5 2024