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

Add vote for articles #7

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

Add vote for articles #7

wants to merge 2 commits into from

Conversation

bugsz
Copy link

@bugsz bugsz commented Apr 23, 2017

Test case is not ready.Will be updated.

@lzjun567
Copy link
Owner

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

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

@bugsz
Copy link
Author

bugsz commented Apr 24, 2017

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

@bugsz
Copy link
Author

bugsz commented Apr 25, 2017

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

@lzjun567
Copy link
Owner

没关系,你有时间再弄

@bugsz
Copy link
Author

bugsz commented Apr 29, 2017

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

@lzjun567
Copy link
Owner

什么token验证?有截图吗

@bugsz
Copy link
Author

bugsz commented Apr 29, 2017

等下发给你

@bugsz
Copy link
Author

bugsz commented Apr 30, 2017

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

@bugsz
Copy link
Author

bugsz commented Apr 30, 2017

不对应该是angularjs的异步

@lzjun567
Copy link
Owner

我这不会啊

@bugsz
Copy link
Author

bugsz commented Apr 30, 2017

你那边没有403?

@lzjun567
Copy link
Owner

lzjun567 commented May 1, 2017

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

@bugsz
Copy link
Author

bugsz commented May 1, 2017

加微信讲吧?很奇怪

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