search

csonchen / wxParse

微信小程序富文本解析
https://csonchen.github.io/wxParse/
MIT License
274 stars 41 forks source link

bindData函数的问题 #5

Closed vito-97 closed 4 years ago

vito-97 commented 4 years ago

`const bindData = (() => { let instance = null return function(bindName, data) { if (!instance) { instance = {

  } 
}else if(data){
  instance[bindName] = data;
}
return instance[bindName]

} })();` 解决了该函数不会进行数据更新，导致图片预览失败

csonchen commented 4 years ago

@vitoSir 你去提个pr吧，我帮你合并一下

csonchen commented 4 years ago

我已经合并了哈，谢谢了，不过有一点要注意一下，commit信息如果是修复bug的话，建议加上前缀：fixBug哈，有可能我要做归并 @vitoSir

vito-97 commented 4 years ago

GitHub也是第一次用，pr还是百度来怎么操作的，见笑了

------------------ 原始邮件 ------------------ 发件人: "csonchen"<notifications@github.com>; 发送时间: 2020年6月18日(星期四) 下午4:34 收件人: "csonchen/wxParse"<wxParse@noreply.github.com>; 抄送: "vito"<775113283@qq.com>;"Mention"<mention@noreply.github.com>; 主题: Re: [csonchen/wxParse] bindData函数的问题 (#5)

我已经合并了哈，谢谢了，不过有一点要注意一下，commit信息如果是修复bug的话，建议加上前缀：fixBug哈，有可能我要做归并 @vitoSir

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

csonchen commented 4 years ago

@vitoSir 贴一下你的html文本给我，你的修复方法有问题，我先暂时去掉了

vito-97 commented 4 years ago

@vitoSir 贴一下你的html文本给我，你的修复方法有问题，我先暂时去掉了

你需要有多篇文章才能测试出问题

csonchen commented 4 years ago

@vitoSir 贴一下你的html文本给我，你的修复方法有问题，我先暂时去掉了

你需要有多篇文章才能测试出问题

你去看一下最上面的一个issue，提供一下有问题的富文本给我，我去看一下