Fetching and rendering of AreWeHeadlessYetTopicPage

wagtail / areweheadlessyet

Are we headless yet?

https://areweheadlessyet.wagtail.org

MIT License

19 stars 4 forks source link

Fetching and rendering of AreWeHeadlessYetTopicPage #5

Closed Tijani-Dia closed 2 years ago

Tijani-Dia commented 2 years ago

There is a topics section in the homepage where is listed all the topics relevant to headless builds with Wagtail. We also have a dedicated topic page which provides more details about each topic. The goal of this PR is to fetch the topic pages from the backend in wagtail.org and render them (Ticket 19 on CodeBase) i.e build the dedicated topic page.

The work done here is based on what have been done in the sections branch hence the target. That also explains why the CI fails; fixing it in that branch will get it resolved here too.

thibaudcolas commented 2 years ago

@Tijani-Dia I don’t think this is reviewable at this stage – as I said in #1, please make sure to always describe the changes you’re making in the pull request description. Here I’d expect you to say:

What the PR is trying to achieve overall
Which tickets it’s for
How you’ve tested it
Why it’s pointing at the sections branch (what is it for?) rather than main? What’s the plan to merge what is inside section?
Why the CI builds are failing / should we ignore this or raise it with you / what’s your plan to get them passing.

Tijani-Dia commented 2 years ago

@thibaudcolas I've added a description to this PR. Let me know if you want more details.

I've tested it locally and in the preview environment.