Skip to content

Login Support#6

Open
dolfelt wants to merge 7 commits into
masterfrom
feature/real-login
Open

Login Support#6
dolfelt wants to merge 7 commits into
masterfrom
feature/real-login

Conversation

@dolfelt

@dolfelt dolfelt commented Aug 27, 2015

Copy link
Copy Markdown
Owner

Add login support using email and password.

Still needs an account picker, otherwise it just defaults to the first one.

Depends on #5.

@mvanhorne

Copy link
Copy Markdown

Just implement jwt token auth with spark and we have all the things

Comment thread src/component/data.js

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Is this sort of a generic cache variable?

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

I was thinking about how to store data in memory. And perhaps in WebSQL or LocalStorage. This was a lame attempt at structuring that.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

👍 Makes sense.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I would strongly recommend using localForage as a wrapper around storage.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

👍 I'll look into it. I haven't had time to even search for one yet.

@dolfelt

dolfelt commented Aug 31, 2015

Copy link
Copy Markdown
Owner Author

Tons of work on this. I feel like I finally figured out a good way to structure the app... for the most part.

Comment thread sass/layout/_forms.scss

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I thought you were going to add an autoprefix plugin for this?

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

I most certainly did! Nice catch. 🏆

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