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

fix(plugin): layers would not be marshalled correctly #126

Merged
merged 2 commits into from
Nov 9, 2021

Conversation

rot1024
Copy link
Member

@rot1024 rot1024 commented Nov 9, 2021

Fix plugin layer APIs would not work as expected. Layer objects would be undefined in the plugin runtime.

@netlify
Copy link

netlify bot commented Nov 9, 2021

✔️ Deploy Preview for reearth-web ready!

🔨 Explore the source changes: bf50063

🔍 Inspect the deploy log: https://app.netlify.com/sites/reearth-web/deploys/618a072578a17b00071ffaaf

😎 Browse the preview: https://deploy-preview-126--reearth-web.netlify.app

@codecov
Copy link

codecov bot commented Nov 9, 2021

Codecov Report

Merging #126 (bf50063) into main (7c65d0c) will decrease coverage by 0.27%.
The diff coverage is 20.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #126      +/-   ##
==========================================
- Coverage   44.81%   44.54%   -0.28%     
==========================================
  Files          56       56              
  Lines        1100     1109       +9     
  Branches      174      179       +5     
==========================================
+ Hits          493      494       +1     
- Misses        554      562       +8     
  Partials       53       53              
Impacted Files Coverage Δ
src/components/atoms/Plugin/index.tsx 25.00% <ø> (ø)
src/components/molecules/Visualizer/Plugin/api.ts 0.00% <0.00%> (ø)
...rc/components/molecules/Visualizer/Plugin/hooks.ts 0.00% <0.00%> (ø)
...rc/components/molecules/Visualizer/Layers/store.ts 56.00% <50.00%> (-0.25%) ⬇️
src/components/atoms/Plugin/hooks.ts 3.22% <100.00%> (ø)

@rot1024 rot1024 merged commit 886302f into main Nov 9, 2021
@rot1024 rot1024 deleted the fix/plugin-layer-marshalling branch November 9, 2021 08:15
keiya01 pushed a commit that referenced this pull request Apr 25, 2023
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.

2 participants