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

Notification read #73

Closed
wants to merge 10 commits into from
Closed

Notification read #73

wants to merge 10 commits into from

Conversation

zhangsoledad
Copy link

No description provided.

@zhangsoledad
Copy link
Author

#70

@zhangsoledad zhangsoledad changed the title [WIP]Notification read Notification read Feb 26, 2016
@tony612
Copy link
Collaborator

tony612 commented Oct 26, 2016

感觉这个 PR 太大了吧。。 可以拆开吗?
另外关于用 brunch 来构建前端部分,我个人觉得现阶段不太必要,特别是对于 elixir-cn 这样的项目来讲 @jw2013 觉得呢?

@kaichen
Copy link

kaichen commented Oct 26, 2016

@tony612 我是觉得应该紧跟 phoenix 官方的配置,这样对于后面他的更新也好,对其他初学者做为范例也好,都是有好处的。

@zhangsoledad
Copy link
Author

忘了close了 很久以前写的 写得不好 现在没时间弄了

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.

3 participants