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

Need help to use feathers authentication storage in vue vuex #329

Closed
kieusonlam opened this Issue Oct 24, 2016 · 5 comments

Comments

Projects
None yet
3 participants
@kieusonlam
Copy link

kieusonlam commented Oct 24, 2016

I'm trying to intergrate feathers with vue + vuex. I got confused about define local storage for feathers authentication. You guys can take a look at my repo:

https://github.com/kieusonlam/vue-feathers-wepack

This file I created for feathers client
vue-feathers-wepack/client/store/feathers.js
if i change storage to localStorage or window.localStorage i will get this error

window.localStorage is not defined

And here is the store i make for feathers authentication
vue-feathers-wepack/client/store/modules/auth.js

@daffl

This comment has been minimized.

Copy link
Member

daffl commented Oct 24, 2016

That problem might occur if you are running your browser in private or incognito mode. Related: #135

@kieusonlam

This comment has been minimized.

Copy link
Author

kieusonlam commented Oct 25, 2016

My problem is about server side rending, i solved it. Thank you for reply 👍 Good to know that problem too.

@kieusonlam kieusonlam closed this Oct 25, 2016

@andycaramba

This comment has been minimized.

Copy link

andycaramba commented Jun 29, 2017

Hi @kieusonlam
Could you explain how you solved this issue?

@kieusonlam

This comment has been minimized.

Copy link
Author

kieusonlam commented Jun 30, 2017

Hi @andycaramba

I just make it running on client side only by using process.env you can take a look at here.

https://github.com/kieusonlam/vue-feathers-wepack/blob/master/client/store/feathers.js

@andycaramba

This comment has been minimized.

Copy link

andycaramba commented Jun 30, 2017

Thank you @kieusonlam

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.