Support for Wechat applet

[einsqing]

Support for Wechat applet，https://developers.weixin.qq.com/miniprogram/en/dev/

[dplewis]

I see what you are doing now. I added a few comments and questions below.

Can you check the travis build and run npm run lint on your code.

[codecov]

Codecov Report

Merging #874 into master will decrease coverage by 0.1%.
The diff coverage is 40%.

@@            Coverage Diff             @@
##           master     #874      +/-   ##
==========================================
- Coverage   92.14%   92.04%   -0.11%     
==========================================
  Files          54       54              
  Lines        5006     5016      +10     
  Branches     1121     1126       +5     
==========================================
+ Hits         4613     4617       +4     
- Misses        393      399       +6

Impacted Files	Coverage Δ
src/LiveQueryClient.js	85.63% <0%> (-1%)	⬇️
src/Storage.js	94.44% <50%> (-1.71%)	⬇️
src/LocalDatastore.js	98.29% <50%> (-0.42%)	⬇️
src/RESTController.js	81.2% <50%> (-0.43%)	⬇️
src/ParseFile.js	96.93% <50%> (-0.59%)	⬇️

---

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 8b61d04...7ecfd75. Read the comment docs.

[davimacedo]

It looks good to me regarding:

• I am sure it will not break the other builds
• The code is perfectly understandable

I haven't tried to create a WeChat app though, but I will probably give a try soon.

[dplewis]

@einsqing I updated the readme can you have a look? Also if you have time, you can add LocalDatastore support as well.

[einsqing]

@dplewis LocalDatastore has been supported

[dplewis]

Is there anything you want to add or change in the readme like documentation or usage with WeChat?

[einsqing]

@dplewis Wechat minprogram is limited to 2m in size. Generally, we use dist/parse.weapp.min.js directly. You can also require ("parse/weapp"), but you need to turn on building NPM packages

[dplewis]

Since you don’t use wx.Parse can you remove it from the PR?

Can you also update the readme with npm run build and copy /dist/parse.weapp.min.js

[einsqing]

@dplewis Update readme and remove wx.Parse

[dplewis]

@acinader looks good to me. Any comments?