Skip to content

Conversation

@adrian-macuc
Copy link
Collaborator

No description provided.

@adrian-macuc adrian-macuc changed the base branch from master to support/v14.8.x November 23, 2023 14:39
@Directive({
selector: '[uiGridCustomSearch], ui-grid-custom-search',
})
export class UiGridCustomSearchDirective {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Similar blocks of code found in 3 locations. Consider refactoring.

@adrian-macuc adrian-macuc marked this pull request as ready for review January 22, 2024 14:04
@adrian-macuc adrian-macuc force-pushed the feat/uigrid_custom_search branch from b177781 to 12f1aa5 Compare January 22, 2024 14:07
@adrian-macuc adrian-macuc changed the base branch from support/v14.8.x to master January 22, 2024 14:08
} from '@angular/core';

@Directive({
selector: '[uiGridCustomSearch], ui-grid-custom-search',
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what's the usecase pls for ref?

@toxik toxik force-pushed the feat/uigrid_custom_search branch from 12f1aa5 to e4d907a Compare February 5, 2024 10:01
@toxik toxik enabled auto-merge February 5, 2024 10:01
@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit e4d907a and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Duplication 1

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 90.3% (0.0% change).

View more on Code Climate.

@toxik toxik merged commit f1047b8 into master Feb 5, 2024
@toxik toxik deleted the feat/uigrid_custom_search branch February 5, 2024 10:08
gheorghitahurmuz pushed a commit that referenced this pull request Feb 5, 2024
feat(ui-grid): add support for using a custom search
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants