Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MR_支持微信小程序相关修改 #400

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

TProgram
Copy link

@TProgram TProgram commented Aug 21, 2020

为了兼容 小程序前端增加相关接口 https://github.com/TProgram/mall-app-web

  • 微信登陆相关
  • 增加相关统计接口 例如 商品总数/购物车数量等
  • 增加意见反馈功能
  • 微信支付相关 - 未完成

@@ -12,6 +12,6 @@
public interface PortalOrderItemDao {
/**
* 批量插入
*/
*/
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

建议处理一下trailing whitespace

private int gender;
private String nickName;
private String language;
private String city;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

city, province, country 最好包装成为locationInfopromoterId以及merchantId是否也同属于一个上下文?

@@ -0,0 +1,40 @@
//package com.macro.mall.portal.util.wechat;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

无用代码是否需要删除?

import com.macro.mall.mapper.UmsFeedBackMapper;
import com.macro.mall.model.UmsMember;
import com.macro.mall.model.UmsFeedBack;
//import com.macro.mall.model.UmsMemberReceiveAddressExample;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
//import com.macro.mall.model.UmsMemberReceiveAddressExample;
//import com.macro.mall.model.UmsMemberReceiveAddressExample;

没有用到的import最好删掉

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.

None yet

2 participants