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.

Sign up for GitHub

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

Jump to bottom

Add button to kick off login process #16

Open
5 tasks done
cwadrupldijjit opened this issue Apr 4, 2017 · 0 comments
Open
5 tasks done

Add button to kick off login process #16

cwadrupldijjit opened this issue Apr 4, 2017 · 0 comments
Assignees
@cwadrupldijjit
Labels
enhancement
Milestone
GitHub sign-in flow

Comments

@cwadrupldijjit
Copy link
Contributor

cwadrupldijjit commented Apr 4, 2017 •
edited
Loading

Estimated Hours to Complete: 2

Actual Hours to Complete: 3

Requirements:

This issue is considered done when:

  • The methods created in Set up Firebase config #15 are used in the creation of a button
  • The button is placed at the bottom of the form that when clicked will fire the method
  • The user is serialized and stored in the local storage
  • The button is only seen if the user isn't found in local storage
  • Add styles so button doesn't look tacky
@cwadrupldijjit cwadrupldijjit added this to the GitHub sign-in flow milestone Apr 4, 2017
@cwadrupldijjit cwadrupldijjit self-assigned this Apr 11, 2017
cwadrupldijjit pushed a commit to cwadrupldijjit/dashcommunity-blog that referenced this issue Apr 12, 2017
feat(login-button): require login on form
0641bcc 
changes:
  - firebaseHelper.js
    - assign token, user, and credential to auth object
    - add to local storage
    - add getAuthObj function to get and deserialize authObj
  - ComposePage/index.jsx
    - import newly created login button and add logic to change whether it shows the action form or if it shows the login button
  - ComposeFormElements/index.jsx
    - add new login button component to exports
  - login-button.css
    - add styles to make sure that it looks non-gaggy
  - LoginButton.jsx
    - create form where the login button will live and call submit function when login is successful
  - package.json
    - ensure that the browser opens to localhost (so firebase auth will work)

issues addressed:
  - dashcommunity#16

no breaking changes--only adds annoyance to the form, not convenience so far
@cwadrupldijjit cwadrupldijjit mentioned this issue Apr 12, 2017
Closed
riongull pushed a commit that referenced this issue May 5, 2017
@cwadrupldijjit @riongull
[PLEASE REVIEW] #17 Auto submit token (#21)
c4e0d10 
* fix(general): update code for eslint conformance

changes:
  - .babelrc
    - fix quote types to make it conformant JSON
  - .eslintrc
    - update linter to not show errors where errors shouldn't be shown
  - .gitignore
    - begin ignoring the public directory--rely on built code
  - ComposeForm/*
    - beginnings of splitting up FormElements.jsx (will be addressed later)
  - PrefixedLink/index.jsx
    - add component to automatically prefix links in order to avoid having to do it manually
  - gatsby-node.js,
        ComposePage/index.jsx,
        Editor/index.jsx,
        FormElements.jsx,
        GitHubForm.jsx,
        gitHubHelper.js,
        localStorageHelper.js,
        ReadNext/index.jsx,
        SiteLinks/index.jsx,
        SiteNav/index.jsx,
        SitePage/index.jsx,
        SitePost/index.jsx,
        SiteSidebar/index.jsx
    - update files to eslint compliance
    - change repetetive field/element creation into data transformed the way you want
  - public/*
    - remove from git (built)

issues addressed:
  - #14

no breaking changes--UI works exactly as originally implemented

* feat(auth): add firebase auth to site

changes:
  - firebaseHelper.js
    - add helper to manage auth
  - index.jsx
    - create module file to import and export necessary components
  - initializeFirebase.js
    - add file to create the firebase connection
  - gitHubHelpers.js
    - import auth into file to test if it wiill work

issues addressed:
  - #15

no breaking changes; auth not yet fully integrated into the project

* feat(login-button): require login on form

changes:
  - firebaseHelper.js
    - assign token, user, and credential to auth object
    - add to local storage
    - add getAuthObj function to get and deserialize authObj
  - ComposePage/index.jsx
    - import newly created login button and add logic to change whether it shows the action form or if it shows the login button
  - ComposeFormElements/index.jsx
    - add new login button component to exports
  - login-button.css
    - add styles to make sure that it looks non-gaggy
  - LoginButton.jsx
    - create form where the login button will live and call submit function when login is successful
  - package.json
    - ensure that the browser opens to localhost (so firebase auth will work)

issues addressed:
  - #16

no breaking changes--only adds annoyance to the form, not convenience so far

* feat(github): remove token from fields and autofill

changes:
  - add helper functions to deal with username
  - firebase auth helper uses authObj instead of exporting individual members
  - clean up slightly the form elements
  - add username prepopulation to the redux form (dosn't work at the moment)
  - automatically grab the token when submitting the form

issues addressed:
  - #17

no breaking changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cwadrupldijjit cwadrupldijjit

Labels
enhancement
Projects
None yet
Milestone
GitHub sign-in flow
Development

No branches or pull requests
1 participant
@cwadrupldijjit