Problem/Motivation

Since the list of node operations can go very long , it would be great to have a filter called 'Node Filter' that would filter the logs via a particular node type

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comments

Anjali Rathod created an issue. See original summary.

anjali rathod’s picture

Assigned: anjali rathod » Unassigned
Status: Active » Needs review
StatusFileSize
new4.26 KB
anjali rathod’s picture

StatusFileSize
new4.41 KB

Another small change in this patch

ajv009’s picture

Assigned: Unassigned » ajv009
Status: Needs review » Needs work

Should not use $_SESSION directly, instead use DI and call user.tempstore service.

Using Drupal 8 services provides needed abstraction and structure for interacting with a global construct. It's part of an overall architecture that allows developers to build and extend complex applications sustainably.

ajv009’s picture

Assigned: ajv009 » Unassigned
Status: Needs work » Reviewed & tested by the community
StatusFileSize
new10.15 KB

I added the temp store service, rest everything is good.

ajv009’s picture

StatusFileSize
new6.99 KB

Had few coding standard issues.

anjali rathod’s picture

Status: Reviewed & tested by the community » Needs work

Please update the validateForm condition

ajv009’s picture

Ops, I missed it.

ajv009’s picture

Status: Needs work » Needs review
anjali rathod’s picture

Status: Needs review » Reviewed & tested by the community

Works fine ! Thankyou

  • AJV009 authored 521dffa on 2.0.x
    Issue #3221669 by AJV009, Anjali Rathod: Create 'Node Type' filter
    
harivenuv’s picture

Thanks you @AJV009 and @Anjali Rathod, Patch looks fine and I have merged.

harivenuv’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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