Skip to content

Commit 75d05df

Browse files
committed
Merge branch 'release/25.15.0'
2 parents 199f2b9 + 762a92a commit 75d05df

File tree

8 files changed

+24
-12
lines changed

8 files changed

+24
-12
lines changed

CHANGELOG.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,9 @@ All notable changes to this project will be documented in this file.
44
The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
55
and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
66

7+
## [25.15.0] - 2025-08-15
8+
### Added
9+
- Fix search page result count
710

811
## [25.14.0] - 2025-08-01
912
### Added

app/institutions/dashboard/-components/object-list/component-test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ module('Integration | institutions | dashboard | -components | object-list', hoo
7171
assert.dom('[data-test-page-tab="summary"]').exists('Summary tab exists');
7272

7373
// Elements in the top bar are present
74-
assert.dom('[data-test-object-count]').containsText('10 total thingies', 'Object count is correct');
74+
assert.dom('[data-test-object-count]').containsText('10,000+ total thingies', 'Object count is correct');
7575
assert.dom('[data-test-toggle-filter-button]').exists('Filter button exists');
7676
assert.dom('[data-test-customize-columns-button]').exists('Columns button exists');
7777

app/models/index-card-search.ts

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
1+
import { inject as service } from '@ember/service';
12
import Model, { AsyncHasMany, attr, hasMany } from '@ember-data/model';
23
import {Links} from 'jsonapi-typescript';
4+
import Intl from 'ember-intl/services/intl';
35

46
import RelatedPropertyPathModel from './related-property-path';
57
import SearchResultModel from './search-result';
@@ -10,12 +12,12 @@ export interface SearchFilter {
1012
filterType?: string;
1113
}
1214

13-
export const ShareMoreThanTenThousand = 'trove:ten-thousands-and-more';
14-
1515
export default class IndexCardSearchModel extends Model {
16+
@service intl!: Intl;
17+
1618
@attr('string') cardSearchText!: string;
1719
@attr('array') cardSearchFilters!: SearchFilter[];
18-
@attr('string') totalResultCount!: number | typeof ShareMoreThanTenThousand;
20+
@attr totalResultCount!: number | object;
1921
@attr('object') links!: Links;
2022

2123
@hasMany('search-result', { inverse: null })
@@ -47,6 +49,14 @@ export default class IndexCardSearchModel extends Model {
4749
}
4850
return null;
4951
}
52+
53+
get displayCount(): number | string {
54+
return (
55+
typeof this.totalResultCount === 'number'
56+
? this.totalResultCount
57+
: this.intl.t('search.ten-thousand-plus')
58+
);
59+
}
5060
}
5161

5262
declare module 'ember-data/types/registries/model' {

lib/osf-components/addon/components/index-card-searcher/component.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ export default class IndexCardSearcher extends Component<IndexCardSearcherArgs>
2323
debounceTime = this.args.debounceTime || 1000;
2424

2525
@tracked searchResults: SearchResultModel[] = [];
26-
@tracked totalResultCount = 0;
26+
@tracked totalResultCount: number | string = 0;
2727

2828
@tracked relatedProperties?: RelatedPropertyPathModel[] = [];
2929
@tracked booleanFilters?: RelatedPropertyPathModel[] = [];
@@ -71,7 +71,7 @@ export default class IndexCardSearcher extends Component<IndexCardSearcherArgs>
7171
this.nextPageCursor = searchResult.nextPageCursor;
7272
this.prevPageCursor = searchResult.prevPageCursor;
7373
this.searchResults = searchResult.searchResultPage.toArray();
74-
this.totalResultCount = searchResult.totalResultCount;
74+
this.totalResultCount = searchResult.displayCount;
7575

7676
return searchResult;
7777
} catch (error) {

lib/osf-components/addon/components/search-page/component.ts

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ import Store from '@ember-data/store';
1111
import { action } from '@ember/object';
1212
import Media from 'ember-responsive';
1313

14-
import { ShareMoreThanTenThousand } from 'ember-osf-web/models/index-card-search';
1514
import InstitutionModel from 'ember-osf-web/models/institution';
1615
import SearchResultModel from 'ember-osf-web/models/search-result';
1716
import ProviderModel from 'ember-osf-web/models/provider';
@@ -83,7 +82,7 @@ export default class SearchPage extends Component<SearchArgs> {
8382
@tracked relatedProperties?: RelatedPropertyPathModel[] = [];
8483
@tracked booleanFilters?: RelatedPropertyPathModel[] = [];
8584
@tracked page?: string = '';
86-
@tracked totalResultCount?: number | {'@id': string};
85+
@tracked totalResultCount?: number | string;
8786
@tracked firstPageCursor?: string | null;
8887
@tracked prevPageCursor?: string | null;
8988
@tracked nextPageCursor?: string | null;
@@ -263,8 +262,7 @@ export default class SearchPage extends Component<SearchArgs> {
263262
this.nextPageCursor = searchResult.nextPageCursor;
264263
this.prevPageCursor = searchResult.prevPageCursor;
265264
this.searchResults = searchResult.searchResultPage.toArray();
266-
this.totalResultCount = searchResult.totalResultCount?.['@id'] === ShareMoreThanTenThousand ? '10,000+' :
267-
searchResult.totalResultCount;
265+
this.totalResultCount = searchResult.displayCount;
268266
} catch (e) {
269267
this.toast.error(e);
270268
}

mirage/views/search.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -300,7 +300,7 @@ export function cardSearch(_: Schema, request: Request) {
300300
],
301301
},
302302
],
303-
totalResultCount: 10,
303+
totalResultCount: {'@id': 'trove:ten-thousands-and-more'},
304304
},
305305
relationships: {
306306
relatedProperties: {

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "ember-osf-web",
3-
"version": "25.14.0",
3+
"version": "25.15.0",
44
"private": true,
55
"description": "Ember front-end for the Open Science Framework",
66
"homepage": "https://github.com/CenterForOpenScience/ember-osf-web#readme",

translations/en-us.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -602,6 +602,7 @@ search:
602602
search-help-label: 'Help tutorial'
603603
total-results: '{count} {count, plural, one {result} other {results} }'
604604
no-results: 'No results found'
605+
ten-thousand-plus: '10,000+'
605606
resource-type:
606607
search-by: 'Search by object type'
607608
all: 'All'

0 commit comments

Comments
 (0)