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

#1159067038 Unauthenticated User Should Not Save A Game #17

Closed
wants to merge 1 commit into from

Conversation

kenware
Copy link
Contributor

@kenware kenware commented Jul 16, 2018

What does this PR do?

It makes sure unauthenticated user does not make an API call to save a game

Description of Task to be completed?

Instead of returning status code 401 if a user is not signed In, an unauthenticated user should not make an API call

How should this be manually tested?

https://et-cfh-staging-pr-17.herokuapp.com/#!/app
Navigate to staging branch on Heroku, start a game when you are not logged in, There will be no API call to save a game

What are the relevant pivotal tracker stories?

#159067038 Unauthenticated user should not save a game

Any background context you want to add?

Screenshots

@adetola-ralph adetola-ralph temporarily deployed to et-cfh-staging-pr-17 July 16, 2018 13:18 Inactive
@kenware kenware changed the title #1159067038 Bug fix Unauthenticated User Should Not Make An API Call To A Game API endpoint #1159067038 Bug fix Unauthenticated User Should Not Save A Game Jul 16, 2018
@adetola-ralph adetola-ralph temporarily deployed to et-cfh-staging-pr-17 July 16, 2018 16:08 Inactive
@adetola-ralph adetola-ralph temporarily deployed to et-cfh-staging-pr-17 July 16, 2018 16:12 Inactive
@kenware kenware changed the title #1159067038 Bug fix Unauthenticated User Should Not Save A Game #1159067038 Unauthenticated User Should Not Save A Game Jul 17, 2018
-modify auth.js
-modify angular controll and places conditional statement
- Makes API call if a user is signed In
[Finishes #159067038]
@coveralls
Copy link

Pull Request Test Coverage Report for Build 340

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.02%) to 66.872%

Totals Coverage Status
Change from base Build 336: -0.02%
Covered Lines: 829
Relevant Lines: 1109

💛 - Coveralls

@kenware kenware closed this Jul 19, 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.

None yet

3 participants