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.

Sign up for GitHub

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

Jump to bottom

证书号的需求描述 #31

Open
lovepoem opened this issue Mar 1, 2020 · 6 comments
Open

证书号的需求描述 #31

lovepoem opened this issue Mar 1, 2020 · 6 comments
Labels
disscussion-wanted disscussion-wanted

Comments

@lovepoem
Copy link
Collaborator

lovepoem commented Mar 1, 2020

描述功能要求

证书上有号码，需要详细描述
@zhaofeng-shu33
Copy link
Member

what do you mean?

@yangli-os
Copy link
Collaborator

证书编号：wuhan2020 + 组别编号 + 人员编号
组别编号2位：01 技术组 ； 02 信息组 ； 03 协调组 ；04 公众号机器人；05 宣传组 06 翻译组；07 渠道组；08 风控组；09 海外团队；
人员编号4位：按照志愿者证书名单.xlsx中的顺序从0001开始编编号。
示例：李扬，协调组-表单上第一个
wuhan2020030001

@lovepoem
Copy link
Collaborator Author

lovepoem commented Mar 3, 2020

这个目前由@16647615268 在疏忽导入时候实现

@lovepoem lovepoem closed this as completed Mar 3, 2020
lovepoem pushed a commit that referenced this issue Mar 8, 2020
@zhaofeng-shu33 @Egria
Generalization (#86)
2955af0 
* move name and website to config file (#3)

* move name and website to config file

* second send email frontend_url abstraction

* move 5 drawing magic number to config

* update sample number

* improve docs (#4)

* improve docs

* re-add add new names

* some format

* [WIP]add frontend code (#5)

* add frontend code

* add logical code

* make a local config file

* add empty css

* add frontend ci

* update path

* use echo

* use data-source

* upload bundle.js

* fix yml syntax

* add frontend docs

* bind this

* tuning doc

* remove .DS_Store

* add dist html template (#6)

* add new api for adding new people

* add test for add new user

* slight change in test file

* save image web api (#79) (#8)

* Update app.py

* Update pic_email.py

* Update test.py

* Update test.py

Co-authored-by: Egria <ned1_32@sina.com>

* add upload image endpoint

* add test for upload image

* fix write image bug

* add missing sample images

* add api doc

* customized EmailDuplicateException (#11)

* customized EmailDuplicateException

* trigger customized EmailDuplicateException

Co-authored-by: 赵丰 (Zhao Feng) <616545598@qq.com>

* switch between normal volunteers and admin

* add option test to increase code coverage

* add frontend switch for two pages

* Implement confirm_admin_token() to be tested

* Update model.py (#12)

Co-authored-by: 赵丰 (Zhao Feng) <616545598@qq.com>

* add admin token in config file

* fix admin_token with utils.get_org_config()

* add token input

* Update test.py

* update admin.js

* test return code

* Update test.py

* update org config (#18)

* Update utils.py

* Update utils.py

* Update app.py

* Update test.py

* Update app.py

* Update app.py

* Update app.py

Co-authored-by: 赵丰 (Zhao Feng) <616545598@qq.com>

* handle token changes (#17)

* handle token changes

* add handle event

* fix classname and target files

* update status

* finish frontend webapi

* fix frontend several problems

Co-authored-by: Egria <ned1_32@sina.com>

* ignore org.json

* add token test

* update readme

* add admin input forms (#23)

* add admin required information

* add form data collect

* add corresponding info

* fix method problem

* fix typo (#22)

* add scss files (#21)

* add scss files

* add missing dev-dep

* add scss contents

* fix data-source reference

* bind resize function

* fix upload image problem

* add volunteer emails (#24)

* change add people api to accept list instead

* add email check in backend

* fix typo

* add returned succ info

* backend add missing contents (#25)

* add name_horizontal_key; add updateOrgConfig api

* add get smtp url

* import right function

* import right module

* add trigger send email (#26)

* add trigger send email

* wrapper a job to a class

* add start send email job api

* import jobs

* use new api in send email job

* change web api name

* add send-email button

* add /api/email test

* add wrong token test

* fix key error

* fix test error

* support not write serial number and date on the template (#28)

* support not write serial number and date on the template

* update example config files

* update image path to image file

* add update_email_config (#27)

* add update_email_config

* format code

* Update app.py

* Update test.py

* change update config api

* Update app.py

* Update test.py

* update api key

* empty string not update

Co-authored-by: 赵丰 (Zhao Feng) <616545598@qq.com>

* add frontend_url and smtp address on the fly (#29)

* add frontend_url and smtp address on the fly

* test inserted key

* wrapper get_smtp_url

* return uniform response (#30)

* change input type to number (#31)

* change input type to number

* use parseint to convert

* create images dir (#32)

* create images dir

* fix message domain

* change start email button type to common button

* bug fix (#35)

* add user doc (#34)

* add user doc

* add user mode pic

Co-authored-by: Egria <ned1_32@sina.com>
@zhaofeng-shu33 zhaofeng-shu33 reopened this Mar 8, 2020
@zhaofeng-shu33
Copy link
Member

It is hard to insert new numbers according to this rule.

@zhaofeng-shu33 zhaofeng-shu33 added the disscussion-wanted disscussion-wanted label Mar 8, 2020
@zhaofeng-shu33
Copy link
Member

I suggest the certificate number can have the uniform form: wuhan202001+ 4 digit number. The 4 digit number is obtained by increasing the id in the database.

@yangli-os
Copy link
Collaborator

如果是通用化的实现，添加的字段应该是要包括group,name,email,number的，这是输入，如果是别的项目的话，更改入口就可以了。别的项目的证书领取时应该也会统计group信息的。就是可以设置为非必选

@zhaofeng-shu33 zhaofeng-shu33 mentioned this issue Mar 26, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
disscussion-wanted disscussion-wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@lovepoem @zhaofeng-shu33 @yangli-os