Skip to content

Display combobox for filtered values and use "direct match" instead of "search"#48

Open
pastakhov wants to merge 28 commits into
SemanticMediaWiki:masterfrom
pastakhov:BrowseData-MinValuesForComboBox
Open

Display combobox for filtered values and use "direct match" instead of "search"#48
pastakhov wants to merge 28 commits into
SemanticMediaWiki:masterfrom
pastakhov:BrowseData-MinValuesForComboBox

Conversation

@pastakhov

Copy link
Copy Markdown
Contributor

Allow to hide the main result column by 'mainlabel=-' parameter
Comment thread includes/Specials/BrowseData/GetApplicableFilters.php Outdated
@pastakhov

Copy link
Copy Markdown
Contributor Author

this branch is deployed on https://wikirouge.net/texts/en/Special:BrowseData/Text

@pastakhov pastakhov marked this pull request as ready for review March 2, 2023 20:34
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #48 (ba07aa6) into master (c517061) will decrease coverage by 0.02%.
The diff coverage is 91.83%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff              @@
##             master      #48      +/-   ##
============================================
- Coverage     68.98%   68.96%   -0.02%     
- Complexity      740      743       +3     
============================================
  Files            36       36              
  Lines          2086     2101      +15     
============================================
+ Hits           1439     1449      +10     
- Misses          647      652       +5     
Impacted Files Coverage Δ
includes/AppliedFilter.php 63.04% <ø> (ø)
includes/Specials/BrowseData/DrilldownQuery.php 100.00% <ø> (ø)
...ludes/Specials/BrowseData/GetApplicableFilters.php 77.36% <91.48%> (-0.50%) ⬇️
includes/Specials/BrowseData/UrlService.php 82.69% <100.00%> (-0.33%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@pastakhov

Copy link
Copy Markdown
Contributor Author

@gesinn-it-gea is it ok?

@gesinn-it-gea

Copy link
Copy Markdown
Member

Many thanks for your contributions! We will review your changes as soon as possible.

@krabina

krabina commented Jul 4, 2023

Copy link
Copy Markdown
Contributor

any news on this, @gesinn-it-gea?

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.

4 participants