Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tree needs to be an array for JSX #17

Merged
merged 1 commit into from
Feb 4, 2017
Merged

Tree needs to be an array for JSX #17

merged 1 commit into from
Feb 4, 2017

Conversation

maraisr
Copy link
Contributor

@maraisr maraisr commented Feb 4, 2017

With the follow example using JSX the code will fail, hence the need to cast the tree argument as an array if not already.

import { h, app } from 'hyperapp';

const model = {};

const update ={};

const view = (model, msg) => (<div><h1>Test</h1></div>);

app({model, view, update});

@jorgebucaran
Copy link
Owner

Thanks @maraisr! 🚀

@jorgebucaran jorgebucaran merged commit 0dd7d5c into jorgebucaran:master Feb 4, 2017
@jorgebucaran
Copy link
Owner

jorgebucaran pushed a commit that referenced this pull request Feb 4, 2017
jorgebucaran added a commit that referenced this pull request Jan 7, 2018
Tree needs to be an array for JSX.
jorgebucaran added a commit that referenced this pull request Jan 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants