Skip to content
This repository has been archived by the owner on Mar 7, 2024. It is now read-only.

fix: 微信组件未声明模板深度导致 base.wxml 过大的问题 #1628

Merged
merged 3 commits into from
Jun 26, 2021

Conversation

noyobo
Copy link
Contributor

@noyobo noyobo commented Jun 25, 2021

新加的这里组件都是 1 深度,默认值是 10

<%_ var len = depth[component.id] || 10; -%>
整体增加了 45kb 左右。

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jun 25, 2021

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 9a42ace:

Sandbox Source
one Configuration

@noyobo noyobo requested review from Darmody and yesmeck June 25, 2021 09:03
@codecov
Copy link

codecov bot commented Jun 25, 2021

Codecov Report

Merging #1628 (9a42ace) into master (87d114d) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1628   +/-   ##
=======================================
  Coverage   90.79%   90.79%           
=======================================
  Files         380      380           
  Lines        4498     4498           
  Branches      648      648           
=======================================
  Hits         4084     4084           
  Misses        406      406           
  Partials        8        8           
Impacted Files Coverage Δ
...i/src/defaultOptions/UNSAFE_wechatTemplateDepth.ts 66.66% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 87d114d...9a42ace. Read the comment docs.

@yesmeck yesmeck merged commit db9484e into master Jun 26, 2021
@yesmeck yesmeck deleted the noyobo-patch-1 branch June 26, 2021 07:56
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.

None yet

2 participants