Skip to content

Conversation

@jitinics
Copy link

  • simple use case
  • grouping module use case
  • grouping module with namespace use case
  • async/await use case

@coveralls
Copy link

coveralls commented Nov 14, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 7337331 on jitinics:master into 329796a on Sellsuki:master.

@n9ti n9ti requested a review from pangaunn November 14, 2017 05:39
@n9ti
Copy link
Contributor

n9ti commented Nov 14, 2017

@pangaunn ถ้าว่างช่วย Review ให้หน่อยนะครับ

@pangaunn
Copy link
Contributor

อันนี้ถามความเห็นนะ ทำไมเราไม่ทำให้ example มันใช้ vue อันเดียวอะ แล้ว example ต่าง ๆ ก็ไปอยู่ใน route เอา
@jitinics @n9ti

Copy link
Contributor

@pangaunn pangaunn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

อยากได้แบบนี้อะ
https://github.com/vuejs/vuex/tree/dev/examples

@n9ti
Copy link
Contributor

n9ti commented Nov 14, 2017

@pangaunn น่าสนใจครับ
แค่ตัว lib เรามันไม่ได้ใช้ vue เลยนะ
แล้วใน package.json จะต้องมี vue ไหมอะ

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.

4 participants