Skip to content
This repository has been archived by the owner on Jan 1, 2024. It is now read-only.

common-actions-auth #34

Closed
dreampod opened this issue Jul 19, 2017 · 1 comment
Closed

common-actions-auth #34

dreampod opened this issue Jul 19, 2017 · 1 comment

Comments

@dreampod
Copy link

dreampod commented Jul 19, 2017

I'm submitting a ...

  • [ x] bug report

  • Is feature important in common use cases?

    If this feature is custom and interesting only for you (not useful for other users, has connection with your app logic, etc.) - email me vladimirmetnew@gmail.com.

  • Describe requested feature:

My project fails build, is this a syntax error?

in src/commons/actions/auth/index.js on line 19 it appears result.data.token isn't wrapped properly.
setLocalToken(result.data.token])

  • What is the expected behavior?

after fixing that, the app compiles correctly.

  • What is the current behavior?

  • Steps to reproduce:

    1. Download from repo

    2. npm install

    3. npm run dev (to see syntax error)

Useful Info:

  • Your environment:

    • Node version: 8.1.4
    • Browser:
    • OS: ubuntu 14 trusty tahr
    • Language/Platform/etc:
  • Other information

    (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, useful links, eg. stackoverflow, gitter, etc)

@Metnew
Copy link
Owner

Metnew commented Jul 19, 2017

Hi, it's a syntax error. Just pull a new version, it should be already fixed.

@Metnew Metnew closed this as completed Jul 19, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants