Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate all React functionality to use @wordpress/element with dependency extraction #2756

Merged
merged 11 commits into from
May 17, 2022
+538 −2,733
Merged

Migrate all React functionality to use @wordpress/element with dependency extraction #2756

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
063b049
Migrate React dependencies to @wordpress/element.
JakePT May 12, 2022
5afff77
Fix dependency name.
JakePT May 12, 2022
fca51b4
Replace third party dependency with core component.
JakePT May 12, 2022
eeb2743
Remove autosuggest polyfill dependencies.
JakePT May 12, 2022
2233d83
Merge branch 'develop' into feature/2613
JakePT May 12, 2022
d704eb7
Merge branch 'develop' into feature/2613
felipeelia May 12, 2022
c509259
Update synonyms test.
JakePT May 13, 2022
561bd22
Merge branch 'feature/2613' of github.com:10up/ElasticPress into feat…
JakePT May 13, 2022
a47f6cb
Merge branch 'develop' into feature/2613
JakePT May 17, 2022
94f2b93
Revert to regular fetch for Autosuggest.
JakePT May 17, 2022
239c83c
Merge branch 'develop' into feature/2613
felipeelia May 17, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Revert to regular fetch for Autosuggest.
  • Loading branch information
@JakePT
JakePT committed May 17, 2022
commit 94f2b9369c3c339763f4b624b9a95e26218a4b1c
19 changes: 9 additions & 10 deletions assets/js/autosuggest.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,10 +1,5 @@
/* eslint-disable camelcase, no-underscore-dangle, no-use-before-define */

/**
* WordPress dependencies.
*/
import apiFetch from '@wordpress/api-fetch';

/**
* Internal dependencies.
*/
Expand Down Expand Up @@ -173,13 +168,11 @@ function buildSearchQuery(searchText, placeholder, { query }) {
async function esSearch(query, searchTerm) {
const fetchConfig = {
body: query,
credentials: 'omit',
method: 'POST',
mode: 'cors',
headers: {
'Content-Type': 'application/json; charset=utf-8',
},
method: 'POST',
mode: 'cors',
url: epas.endpointUrl,
};

if (epas?.http_headers && typeof epas.http_headers === 'object') {
Expand All @@ -194,7 +187,13 @@ async function esSearch(query, searchTerm) {
}

try {
const data = await apiFetch(fetchConfig);
const response = await fetch(epas.endpointUrl, fetchConfig);

if (!response.ok) {
throw Error(response.statusText);
}

const data = await response.json();

// allow for filtered data before returning it to
// be output on the front end
Expand Down