Skip to content
This repository has been archived by the owner on Nov 28, 2023. It is now read-only.

Fix the hello-world example code #45

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

jpc
Copy link

@jpc jpc commented Oct 12, 2021

Running the initial example on the demo page gives a Children must be a list of VDOM objects or strings which is not a great first experience with the project ;)

Python lists of VDOM objects are neither VDOM objects nor strings so they trigger the error. If we splice the results of the fancy call into the h1 arguments list (so each list element becomes a separate argument to h1) then the code works correctly.

Running the initial example on the demo page gives a `Children must be a list of VDOM objects or strings` which is not a great first experience with the project ;)

Python lists of VDOM objects are neither VDOM objects nor strings so they trigger the error. If we splice the results of the `fancy` call into the `h1` arguments list (so each list element becomes a separate argument to `h1`) then the code works correctly.
@vercel
Copy link

vercel bot commented Oct 12, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/nteract/play/DoGfa9PRwXyr6XVieYAZbZDSvLbU
✅ Preview: https://play-git-fork-jpc-patch-1-nteract.vercel.app

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant