This repository has been archived by the owner on Feb 27, 2024. It is now read-only.
Replies: 1 comment
-
@donmhico I'm fine with this change. Submit a PR and I'll get it merged in. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
I noticed that if the Algolia env vars are not set (
NEXT_PUBLIC_ALGOLIA_INDEX_NAME
,NEXT_PUBLIC_ALGOLIA_APPLICATION_ID
,NEXT_PUBLIC_ALGOLIA_SEARCH_ONLY_KEY
) the site doesn't load. I get this errorIn my own opinion, as a starter, I think the lesser we need to setup the better.
We can change this block of code
nextjs-wordpress-starter/api/algolia/connector.js
Lines 4 to 6 in 1ed3c99
to
This change will make the Algolia related env vars optional, at least until the user actually wants to use the Algolia Search component.
Beta Was this translation helpful? Give feedback.
All reactions