search

blitz-js / babel-plugin-superjson-next

Automatically transform your Next.js Pages to use SuperJSON
MIT License
124 stars 15 forks source link

Fix nextjs builds if there are no "meta" values #6

Closed benjamminj closed 3 years ago

benjamminj commented 3 years ago

Resolves #5

Alternatively, we could fix this upstream in superjson if we'd rather do that, although I think that would involve changing meta to default to null if there's no transformed properties.

Skn0tt commented 3 years ago

Thank you for this awesome PR! You even included a test 🥰 I'll gladly merge.

we could fix this upstream in superjson

I think this is the right place, we shouldn't optimise for any Next.js specific things in superjson itself.

Skn0tt commented 3 years ago

@allcontributors add Benjamminj code test bug

allcontributors[bot] commented 3 years ago

@Skn0tt

I've put up a pull request to add @Benjamminj! :tada:

blakeley commented 3 years ago

This fixed an issue I was having -- thanks so much Benjamin!!