This repository was archived by the owner on Mar 4, 2025. It is now read-only.
integration with accounts app for login/reg flows #819
+239
−520
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 30 commits
April 19, 2016 12:27
…coder.com -- Login and Logout using accounts app sso
…coder.com -- Had to remove the kissmeterics scripts because it is creating another iframe which is conflicting with accounts app iframe. Need to do proper fix later on
…coder.com -- Temporary travis build to test out the changes on dev. On local facing problem with accounts app's /tc URL
…coder.com -- Fixing lint errors
* dev: Remove box-shadow on hover from member search field Trigger travis for final member search Trigger travis for final member search Trigger to pick up new member search updates Fix error message
…coder.com -- Fixed unit tests
…coder.com -- Trying out fix for intros.scss extend
…coder.com -- Fixed error in intros.scss extend
…coder.com -- Adapting for changes made for configuring iframe in accounts app
…coder.com -- http => https for connector and accounts app url
…coder.com -- Fixed jwt token service to return the token
…coder.com -- Handling login state of topcoder-app to redirect user to correct path. For logged in users, load state specified by `next` param or go to dashboard and for non logged in user, it redirects user to the login page
…coder.com -- Fixed lint errors
…coder.com -- Added debug statements
…coder.com -- Fixing state.go call to use state name instead of state object
…coder.com -- Debug statements
…coder.com -- Login only for states which requires authentication -- notify state change after login
…coder.com -- Trying with empty view to kick off the controller
…coder.com -- Returned token for v2 and exception calls
…coder.com -- Handled v2 token presence for detecting if user is logged in or not
…coder.com -- Added back the reference to AuthTokenService
* dev: Update url for community page search Fix extend error and introjs button css overrides Add optional for now so we can deploy
…coder.com -- Trying to fix the state.go operation, it is not loading the state right now.
…coder.com -- Using window.location to redirect user instead of using state.go
…coder.com -- Reverted changes for window.location, because it caused infinite redirect
…coder.com -- Trying to fix state.go with url router sync
…coder.com -- Fixing node env error
…coder.com -- Trying to get LoginController executed
…coder.com -- Debug statements
added 12 commits
April 28, 2016 16:17
…coder.com -- Debug statements
…coder.com -- Debug statements
…coder.com -- Debug statements
…coder.com -- preventing state change until the promise is resolved to allow token to be available for all api calls.
…coder.com -- fixed lint errors
…coder.com -- Misc fixes to get it working for pages which don't require auth
…coder.com -- Misc fixes to get it working for pages which don't require auth
* dev: (64 commits) AS#131482348128949, Refactor Mailchimp API wrapper to be generic user preferences service Fixed lint errors Added code comments Removed tabs updated package.json Final fix - all review comments Fix indentation issue Implemented/Fixed review comments, except package.json. Will do that next. Fixed typos Fixed all review feedbacks Fixed left lint errors Fixed lint errors Fixed issues with tooltip Removed extra semicolon Fixed lint errors Fixed review comments Fixed unused var lint errors Fixed lint issues Fixed issue #814 - [$100] Topcoder Member Profile: Redesign ratings graph AS#131482348128949, Refactor Mailchimp API wrapper to be generic user preferences service ... Conflicts: app/topcoder.constants.js
* dev: AS#131482348128949, Refactor Mailchimp API wrapper to be generic user preferences service Conflicts: app/topcoder.constants.js
* dev: AS#131482348128949, Refactor Mailchimp API wrapper to be generic user preferences service Conflicts: app/topcoder.constants.js
…coder.com -- Reduced debug statements
…coder.com -- Removed temporary branch build as changes are being merged into dev now and we already have a build for dev env
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@parthshah @sudoster @ykohata fyi, merging the PR for now to get the changes on dev, however, please feel free to suggest anything that I can improve.