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

健康成就-没有医生,日记排列错乱 #45

Closed
georgeliuyu opened this issue Sep 12, 2018 · 9 comments
Closed

健康成就-没有医生,日记排列错乱 #45

georgeliuyu opened this issue Sep 12, 2018 · 9 comments
Labels
bug Something isn't working 已解决待确认 过期

Comments

@georgeliuyu
Copy link
Contributor

1、这个健康成就是有医生的
2、日记排列是错乱的:顺序是:26/3 15/3 25/3 07/5 01/4

image

健康成就:
image

@rand

@georgeliuyu georgeliuyu added the bug Something isn't working label Sep 12, 2018
@georgeliuyu
Copy link
Contributor Author

@ben7th 问了南希,错乱的问题有提出来过。你组织梳理一下流程,上线前产品是否要review确认一下的。

@nancy919
Copy link

@georgeliuyu 关于医生,所做的处理是,若创建成就有添加医生,则显示,否则不显示;

@ben7th
Copy link
Contributor

ben7th commented Sep 12, 2018

@georgeliuyu 做过 review,这个可能是有遗漏。
目前 review 的流程可能需要更严谨(和 github 流程,分支,pr 结合)来避免遗漏
流程上的改进我来考虑

@zhaobinglong
Copy link

zhaobinglong commented Sep 12, 2018

关于医生:

前端确认用户的确编辑了医生,但是医生是在字段doctor中,而新版本采用的是doctors字段,这个成就是在第一版时创建的,所以这里可能需要做一次数据迁移,将数据库中doctor中的数据,迁移进入doctors

image

前端在用户界面再做一次兼容处理

@zhaobinglong
Copy link

日记排列问题已确认,协调后端排查具体原因

@zhaobinglong
Copy link

医生问题已fix,现在单独解决排序问题

@zhaobinglong
Copy link

关于成就的开始时间和结束时间:

image

老版本的这两个时间是用户主动输入的,而新版本的这两个时间,是用户在编辑健康成就时,根据日记的最早和最晚时间计算的,所以,如果一个老版本的健康成就,没有在新版本再次编辑过,它的开始和结束时间,还是以前用户输入的时间,并非日记的最早和最晚时间

@stale
Copy link

stale bot commented Oct 8, 2018

这个 issue 已经至少十五天没有任何回复了,所以被自动标记为“过期”。 如果再过十五天没有回复,这个 issue 就会被自动关闭。 建议尽快处理。

@stale stale bot added the 过期 label Oct 8, 2018
@stale
Copy link

stale bot commented Oct 23, 2018

这个 issue 已经至少三十天没有任何回复了,被自动关闭。

@stale stale bot closed this as completed Oct 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working 已解决待确认 过期
Projects
None yet
Development

No branches or pull requests

4 participants