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

#167970785 Implement redirection to frontend (social) #73

Merged
merged 1 commit into from
Aug 21, 2019

Conversation

codeBlock-1984
Copy link
Contributor

What does this PR do?

  • Implements redirection to the Authors Haven frontend for social login

Description of Task to be completed?

  • Implement redirection to frontend URL
  • Change the backend port to prevent conflict

How should this be manually tested?

Any background context you want to provide?

What are the relevant pivotal tracker stories?

Screenshots (if appropriate)

Questions:

@@ -48,6 +49,7 @@ describe('GET Social Login', () => {
};
sinon.stub(User, 'findOrCreate').returns(dbResponse);
await socialController.socialUser(request, response);
console.log(response);
Copy link

Choose a reason for hiding this comment

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

Unexpected console statement no-console

@@ -38,6 +38,7 @@ describe('GET Social Login', () => {
};
sinon.stub(User, 'findOrCreate').returns(dbResponse);
await socialController.socialUser(request, response);
console.log(response);
Copy link

Choose a reason for hiding this comment

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

Unexpected console statement no-console

@czarjulius czarjulius temporarily deployed to ah-nyati-backend-staging-pr-73 August 21, 2019 11:18 Inactive
- Implement redirection to frontend url
- Change backend port to prevent conflict

[Delivers #167970785]
@fob413 fob413 merged commit b8245e8 into staging Aug 21, 2019
@fob413 fob413 deleted the bg-redirection-social-callback-167970785 branch August 21, 2019 11:53
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.

None yet

7 participants