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

Example using VueJS #138

Merged
merged 7 commits into from
Mar 6, 2018
Merged

Example using VueJS #138

merged 7 commits into from
Mar 6, 2018

Conversation

itrew
Copy link
Contributor

@itrew itrew commented Mar 1, 2018

Wanted to get my feet wet with this lib and thought it would be helpful to contribute in the process. This is a VueJS app that uses the authentication and request packages.

Initial template created with webpack-simple then added on the vue-router plugin to handle the authentication callbacks.

image

itrew added 2 commits March 1, 2018 10:46
Working demo with and without popups.

Minor tweaks.
@coveralls
Copy link

coveralls commented Mar 1, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling cf4acca on itrew:vue-demo into 1173a88 on Esri:master.

@jgravois
Copy link
Contributor

jgravois commented Mar 1, 2018

rock and roll 🎼! it looks like this even resolves #100.

@itrew
Copy link
Contributor Author

itrew commented Mar 1, 2018

Aside from #11, #100 was in my mind when I was creating this. This type of implementation would require some form of routing framework to work, though.

I'm not entirely sure how to pass a session through a redirect like you mentioned. Perhaps a localStorage implementation? I can mess around with that a bit and see what I find.

@patrickarlt
Copy link
Contributor

@itrew do you want to update this with the localstorage solution from #140 and then I can review it?

@itrew
Copy link
Contributor Author

itrew commented Mar 3, 2018

@patrickarlt Thanks for the feedback on #140, everything makes sense 👍 I'll spend some time in the next couple of days updating this with both your's and @jgravois 's input on this.

Copy link
Contributor

@patrickarlt patrickarlt left a comment

Choose a reason for hiding this comment

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

Looks great @itrew! I'm going to to hold off on merging this so @jgravois can have a look too.

@jgravois
Copy link
Contributor

jgravois commented Mar 6, 2018

i agree. thanks for both your contributions and for commenting your code so thoroughly!

@jgravois jgravois merged commit 509afd7 into Esri:master Mar 6, 2018
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