Skip to content

Commit d59dc4f

Browse files
Linda PengLinda Peng
authored andcommitted
Refactor /resources page to use React Query with search
1 parent 0675010 commit d59dc4f

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

src/components/Resources/index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ import { getResources } from '../../utils/queries';
1010
function Resources() {
1111
const [searchValue, setSearchValue] = useState('');
1212
const { isLoading, data, isError, error } = useQuery(
13-
['resource', searchValue],
13+
[searchValue],
1414
getResources
1515
);
1616

src/components/Resources/index.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ jest.mock('axios');
1414
// search after clicking on search button
1515
describe('Resources', () => {
1616
test('renders correctly with resources', async () => {
17-
const url = '/api/v1/resources';
17+
const url = '/api/v1/resources/';
1818

1919
axiosMock.get.mockResolvedValueOnce({
2020
data: {

src/utils/queries.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,11 @@ const getResource = async (_key, id) => {
88
};
99

1010
const getResources = async searchTerm => {
11-
const { data } = await axios.get(
12-
`${API_URL}/resources/?search=${searchTerm}`
13-
);
14-
console.log('get resources!!!!!');
15-
console.log(data);
11+
const url =
12+
searchTerm !== ''
13+
? `${API_URL}/resources/?search=${searchTerm}` // Empty '' search term will return 0 results
14+
: `${API_URL}/resources/`;
15+
const { data } = await axios.get(url);
1616
return data;
1717
};
1818

0 commit comments

Comments
 (0)