Skip to content

Add vote for articles #7

Open
bugsz wants to merge 2 commits into
lzjun567:masterfrom
bugsz:master
Open

Add vote for articles #7
bugsz wants to merge 2 commits into
lzjun567:masterfrom
bugsz:master

Conversation

@bugsz

@bugsz bugsz commented Apr 23, 2017

Copy link
Copy Markdown

Test case is not ready.Will be updated.

@lzjun567

Copy link
Copy Markdown
Owner

@WhatAreYouSaying
master 分支已经更新,可以把代码合并过去后在 pull request
另外

  1. 可以考虑把所有相对路径的import改成绝对路径。
  2. 如果还没有想好单元测试怎么写的时候可以考虑想好之后在PR
  3. 其实单元测试可以只考虑他有没有返回期望的字段即刻

@bugsz

bugsz commented Apr 24, 2017

Copy link
Copy Markdown
Author

好谢谢
这是我第一个项目 请多多指教

@bugsz

bugsz commented Apr 25, 2017

Copy link
Copy Markdown
Author

这几天考试
可能五一节会更

@lzjun567

Copy link
Copy Markdown
Owner

没关系,你有时间再弄

@bugsz

bugsz commented Apr 29, 2017

Copy link
Copy Markdown
Author

这个...突然发现知乎专栏加了个token验证..能帮我看下吗

@lzjun567

Copy link
Copy Markdown
Owner

什么token验证?有截图吗

@bugsz

bugsz commented Apr 29, 2017

Copy link
Copy Markdown
Author

等下发给你

@bugsz

bugsz commented Apr 30, 2017

Copy link
Copy Markdown
Author

图片发不出来,就是直接访问zhuanlan.zhihu.com时候会有一个XSRF-TOKEN,没看到这个请求之前有任何请求,找不到哪里来的。所以总是403

@bugsz

bugsz commented Apr 30, 2017

Copy link
Copy Markdown
Author

不对应该是angularjs的异步

@lzjun567

Copy link
Copy Markdown
Owner

我这不会啊

@bugsz

bugsz commented Apr 30, 2017

Copy link
Copy Markdown
Author

你那边没有403?

@lzjun567

lzjun567 commented May 1, 2017

Copy link
Copy Markdown
Owner

估计是你的请求数据有问题,看看提交的数据格式有没有问题,是表单还是json的

@bugsz

bugsz commented May 1, 2017

Copy link
Copy Markdown
Author

加微信讲吧?很奇怪

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.

2 participants