Skip to content

Commit 2fcf485

Browse files
authored
feat(saveSynonyms): replaceExistingSynonyms -> clearExistingSynonyms (#1226)
fixes #1224
1 parent 6cb1d0a commit 2fcf485

File tree

5 files changed

+22
-6
lines changed

5 files changed

+22
-6
lines changed

packages/algoliasearch/src/builds/browser.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -451,7 +451,10 @@ export type SearchIndex = BaseSearchIndex & {
451451
readonly replaceAllSynonyms: (
452452
synonyms: readonly Synonym[],
453453
requestOptions?: RequestOptions &
454-
Pick<SaveSynonymsOptions, Exclude<keyof SaveSynonymsOptions, 'replaceExistingSynonyms'>>
454+
Pick<
455+
SaveSynonymsOptions,
456+
Exclude<keyof SaveSynonymsOptions, 'clearExistingSynonyms' | 'replaceExistingSynonyms'>
457+
>
455458
) => Readonly<WaitablePromise<SaveSynonymsResponse>>;
456459
readonly searchRules: (
457460
query: string,

packages/algoliasearch/src/builds/node.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -454,7 +454,10 @@ export type SearchIndex = BaseSearchIndex & {
454454
readonly replaceAllSynonyms: (
455455
synonyms: readonly Synonym[],
456456
requestOptions?: RequestOptions &
457-
Pick<SaveSynonymsOptions, Exclude<keyof SaveSynonymsOptions, 'replaceExistingSynonyms'>>
457+
Pick<
458+
SaveSynonymsOptions,
459+
Exclude<keyof SaveSynonymsOptions, 'clearExistingSynonyms' | 'replaceExistingSynonyms'>
460+
>
458461
) => Readonly<WaitablePromise<SaveSynonymsResponse>>;
459462
readonly searchRules: (
460463
query: string,

packages/client-search/src/methods/index/replaceAllSynonyms.ts

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,14 @@ export const replaceAllSynonyms = (base: SearchIndex) => {
99
return (
1010
synonyms: readonly Synonym[],
1111
requestOptions?: RequestOptions &
12-
Pick<SaveSynonymsOptions, Exclude<keyof SaveSynonymsOptions, 'replaceExistingSynonyms'>>
12+
Pick<
13+
SaveSynonymsOptions,
14+
Exclude<keyof SaveSynonymsOptions, 'clearExistingSynonyms' | 'replaceExistingSynonyms'>
15+
>
1316
): Readonly<WaitablePromise<SaveSynonymsResponse>> => {
1417
return saveSynonyms(base)(synonyms, {
1518
...requestOptions,
16-
replaceExistingSynonyms: true,
19+
clearExistingSynonyms: true,
1720
});
1821
};
1922
};

packages/client-search/src/methods/index/saveSynonyms.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,14 +10,15 @@ export const saveSynonyms = (base: SearchIndex) => {
1010
synonyms: readonly Synonym[],
1111
requestOptions?: SaveSynonymsOptions & RequestOptions
1212
): Readonly<WaitablePromise<SaveSynonymsResponse>> => {
13-
const { forwardToReplicas, replaceExistingSynonyms, ...options } = requestOptions || {};
13+
const { forwardToReplicas, clearExistingSynonyms, replaceExistingSynonyms, ...options } =
14+
requestOptions || {};
1415
const mappedRequestOptions = createMappedRequestOptions(options);
1516

1617
if (forwardToReplicas) {
1718
mappedRequestOptions.queryParameters.forwardToReplicas = 1; // eslint-disable-line functional/immutable-data
1819
}
1920

20-
if (replaceExistingSynonyms) {
21+
if (replaceExistingSynonyms || clearExistingSynonyms) {
2122
mappedRequestOptions.queryParameters.replaceExistingSynonyms = 1; // eslint-disable-line functional/immutable-data
2223
}
2324

packages/client-search/src/types/SaveSynonymsOptions.ts

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,12 @@ export type SaveSynonymsOptions = {
66

77
/**
88
* If the existing synonyms should be removed.
9+
* @deprecated use clearExistingSynonyms
910
*/
1011
readonly replaceExistingSynonyms?: boolean;
12+
13+
/**
14+
* If the existing synonyms should be removed.
15+
*/
16+
readonly clearExistingSynonyms?: boolean;
1117
};

0 commit comments

Comments
 (0)