Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add dynamic counts to search filters #5471

Merged
merged 80 commits into from
Nov 13, 2024

Rename consistency

2c4b3cf
Select commit
Loading
Failed to load commit list.
Merged

Add dynamic counts to search filters #5471

Rename consistency
2c4b3cf
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Nov 13, 2024 in 1s

93.30% of diff hit (target 78.54%)

View this Pull Request on Codecov

93.30% of diff hit (target 78.54%)

Annotations

Check warning on line 9 in app/assets/javascripts/components/saved_annotations/saved_annotation_list.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/components/saved_annotations/saved_annotation_list.ts#L9 

Added line #L9 was not covered by tests

Check warning on line 55 in app/assets/javascripts/components/search/filter_element.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/components/search/filter_element.ts#L55 

Added line #L55 was not covered by tests

Check warning on line 71 in app/assets/javascripts/components/search/search_token.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/components/search/search_token.ts#L70-L71 

Added lines #L70 - L71 were not covered by tests

Check warning on line 75 in app/assets/javascripts/components/search/search_token.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/components/search/search_token.ts#L73-L75 

Added lines #L73 - L75 were not covered by tests

Check warning on line 98 in app/assets/javascripts/components/search/search_token.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/components/search/search_token.ts#L98 

Added line #L98 was not covered by tests

Check warning on line 100 in app/assets/javascripts/state/SavedAnnotations.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/assets/javascripts/state/SavedAnnotations.ts#L100 

Added line #L100 was not covered by tests

Check warning on line 2 in app/controllers/events_controller.rb

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/events_controller.rb#L2 

Added line #L2 was not covered by tests

Check warning on line 4 in app/controllers/events_controller.rb

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/events_controller.rb#L4 

Added line #L4 was not covered by tests

Check warning on line 8 in app/controllers/events_controller.rb

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/events_controller.rb#L8 

Added line #L8 was not covered by tests

Check warning on line 47 in app/controllers/feedbacks_controller.rb

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/feedbacks_controller.rb#L47 

Added line #L47 was not covered by tests

Check warning on line 50 in app/controllers/feedbacks_controller.rb

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/feedbacks_controller.rb#L49-L50 

Added lines #L49 - L50 were not covered by tests

Check warning on line 72 in app/models/concerns/filterable.rb

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/models/concerns/filterable.rb#L72 

Added line #L72 was not covered by tests
View more details on Codecov