Skip to content

Commit 0ba0de8

Browse files
authored
Merge pull request Expensify#67114 from Expensify/cmartins-makeSearchRead
[No QA] Make search a read command
2 parents 1541f78 + d84d63a commit 0ba0de8

3 files changed

Lines changed: 4 additions & 4 deletions

File tree

src/libs/API/types.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,6 @@ const WRITE_COMMANDS = {
9494
SIGN_IN_WITH_GOOGLE: 'SignInWithGoogle',
9595
SIGN_IN_USER: 'SigninUser',
9696
SIGN_IN_USER_WITH_LINK: 'SigninUserWithLink',
97-
SEARCH: 'Search',
9897
REQUEST_UNLINK_VALIDATION_LINK: 'RequestUnlinkValidationLink',
9998
UNLINK_LOGIN: 'UnlinkLogin',
10099
ENABLE_TWO_FACTOR_AUTH: 'EnableTwoFactorAuth',
@@ -735,7 +734,6 @@ type WriteCommandParameters = {
735734
[WRITE_COMMANDS.REMOVE_POLICY_CATEGORY_RECEIPTS_REQUIRED]: Parameters.RemovePolicyCategoryReceiptsRequiredParams;
736735
[WRITE_COMMANDS.SET_POLICY_CATEGORY_MAX_AMOUNT]: Parameters.SetPolicyCategoryMaxAmountParams;
737736
[WRITE_COMMANDS.SET_POLICY_CATEGORY_APPROVER]: Parameters.SetPolicyCategoryApproverParams;
738-
[WRITE_COMMANDS.SEARCH]: Parameters.SearchParams;
739737
[WRITE_COMMANDS.SET_POLICY_CATEGORY_TAX]: Parameters.SetPolicyCategoryTaxParams;
740738
[WRITE_COMMANDS.JOIN_POLICY_VIA_INVITE_LINK]: Parameters.JoinPolicyInviteLinkParams;
741739
[WRITE_COMMANDS.GET_ACCESSIBLE_POLICIES]: null;
@@ -1020,6 +1018,7 @@ const READ_COMMANDS = {
10201018
OPEN_PLAID_CARDS_BANK_LOGIN: 'OpenPlaidCardsBankLogin',
10211019
OPEN_PLAID_BANK_ACCOUNT_SELECTOR: 'OpenPlaidBankAccountSelector',
10221020
OPEN_SEARCH_PAGE: 'OpenSearchPage',
1021+
SEARCH: 'Search',
10231022
GET_OLDER_ACTIONS: 'GetOlderActions',
10241023
GET_NEWER_ACTIONS: 'GetNewerActions',
10251024
EXPAND_URL_PREVIEW: 'ExpandURLPreview',
@@ -1111,6 +1110,7 @@ type ReadCommandParameters = {
11111110
[READ_COMMANDS.OPEN_INITIAL_SETTINGS_PAGE]: null;
11121111
[READ_COMMANDS.OPEN_ENABLE_PAYMENTS_PAGE]: null;
11131112
[READ_COMMANDS.OPEN_SEARCH_PAGE]: null;
1113+
[READ_COMMANDS.SEARCH]: Parameters.SearchParams;
11141114
[READ_COMMANDS.BEGIN_SIGNIN]: Parameters.BeginSignInParams;
11151115
[READ_COMMANDS.SIGN_IN_WITH_SHORT_LIVED_AUTH_TOKEN]: Parameters.SignInWithShortLivedAuthTokenParams;
11161116
[READ_COMMANDS.SIGN_IN_WITH_SUPPORT_AUTH_TOKEN]: Parameters.SignInWithSupportAuthTokenParams;

src/libs/Network/NetworkStore.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ function getAuthToken(): string | null | undefined {
100100
function isSupportRequest(command: string): boolean {
101101
return [
102102
WRITE_COMMANDS.OPEN_APP,
103-
WRITE_COMMANDS.SEARCH,
103+
READ_COMMANDS.SEARCH,
104104
WRITE_COMMANDS.UPDATE_NEWSLETTER_SUBSCRIPTION,
105105
WRITE_COMMANDS.OPEN_REPORT,
106106
SIDE_EFFECT_REQUEST_COMMANDS.RECONNECT_APP,

src/libs/actions/Search.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -297,7 +297,7 @@ function search({
297297
};
298298
const jsonQuery = JSON.stringify(query);
299299

300-
API.write(WRITE_COMMANDS.SEARCH, {hash: queryJSON.hash, jsonQuery}, {optimisticData, finallyData, failureData});
300+
API.read(READ_COMMANDS.SEARCH, {hash: queryJSON.hash, jsonQuery}, {optimisticData, finallyData, failureData});
301301
}
302302

303303
/**

0 commit comments

Comments
 (0)