Skip to content

Conversation

@junhoyeo
Copy link
Contributor

@junhoyeo junhoyeo commented Dec 4, 2022

  1. Since target is ReflectSceneNode, target.fills can be empty, which can sometimes lead to crash.
  2. Also, we're using multiple .filter(...)/.map(...) chains. This can be combined into .flatMap(...).
  3. This PR also changes type of images to match parameter typings for service.fetch(...): (string | null)[] -> string[]

Screenshot 2022-12-04 at 11 10 31 PM

@vercel
Copy link

vercel bot commented Dec 4, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
code ✅ Ready (Inspect) Visit Preview Dec 4, 2022 at 3:14PM (UTC)

@softmarshmallow
Copy link
Member

👍 Good points

@softmarshmallow softmarshmallow merged commit b1d1137 into main Dec 4, 2022
@junhoyeo junhoyeo deleted the junhoyeo/empty-fills branch December 4, 2022 21:39
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