Skip to content

Support empty target.fills when resolving image hashes#195

Merged
softmarshmallow merged 2 commits intomainfrom
junhoyeo/empty-fills
Dec 4, 2022
Merged

Support empty target.fills when resolving image hashes#195
softmarshmallow merged 2 commits intomainfrom
junhoyeo/empty-fills

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