search

wagtail / areweheadlessyet

Are we headless yet?
https://areweheadlessyet.wagtail.org
MIT License
19 stars 4 forks source link

Fetching and rendering of AreWeHeadlessYetHomePage #4

Closed Tijani-Dia closed 2 years ago

Tijani-Dia commented 2 years ago

The goal of this PR is to fetch the AreWeHeadlessYet Homepage and render it.

I have added configuration files for staging and production instance of wagtail.org. However the work done on wagtail.org hasn't been merged yet and is only available on staging so we can't make queries to wagtail.org yet. This is what makes the CI jobs fail.

I changed the production environment variables file to point to the staging instance but it's missing some env. variables (AUTH_USER and AUTH_PASSWORD) - I have added those in Vercel which is why the Vercel job succeeds.

thibaudcolas commented 2 years ago

@Tijani-Dia could you update this based on #1 / merge #1 and then rebase this branch? Additionally this PR is way too big to be comfortably reviewable – for me it makes my browser crash just to try and view the file changes in GitHub. Is there any chance you could split this into smaller PRs?

Re the failing builds – what’s your plan regarding those? Are we holding off from merging this PR until this is resolved? Should we stop running npm run build in CI, since it runs in Vercel anyway?

Tijani-Dia commented 2 years ago

@thibaudcolas I will try to split it in smaller PRs.

Re the failing builds - It depends on whether we are going to deploy the backend to production soon because that would fix this issue. Otherwise stop running npm run build in CI might be our best bet.