Skip to content

feat(ARC-3770): add search param to get themes endpoint, fix tests#704

Merged
bertyhell merged 1 commit into
release/v4.3from
feature/ARC-3770-search-themes
Jul 13, 2026
Merged

feat(ARC-3770): add search param to get themes endpoint, fix tests#704
bertyhell merged 1 commit into
release/v4.3from
feature/ARC-3770-search-themes

Conversation

@bertyhell

@bertyhell bertyhell commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

@bertyhell bertyhell requested a review from reunefe July 13, 2026 09:30
app_material_requests(
where: {
status: { _eq: APPROVED }
_or: [

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hoe is dit erin geraakt? 🤔

@bertyhell bertyhell Jul 13, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

gelieve shift click op rijen te gebruiken in github anders zie ik dit:
Image

die comments """ zijn geldige graphql comments, maar onze type generation script kan daar niet mee overweg. dus ik heb ze terug gewist

const direction = (orderDirection ?? SortDirectionWithRandom.asc) as unknown as Order_By;
const orderBy = [set({}, dbProp, direction)];

if (searchPattern) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Niets ophalen als er geen searchTerm wordt opgegeven?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

nee, de else zit in de rest van de functie:

const response = await this.dataService.execute<GetThemesQuery, GetThemesQueryVariables>(
GetThemesDocument,
{ offset, limit: size, orderBy }
);
const total = response.app_theme_aggregate.aggregate.count;
return Pagination<ThemeResponseDto>({
items: response.app_theme.map((theme) => this.adaptTheme(theme)),
page,
size,
total,
});

@bertyhell bertyhell merged commit 1488448 into release/v4.3 Jul 13, 2026
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.

2 participants