Skip to content
This repository has been archived by the owner on Jan 19, 2021. It is now read-only.

[Fix bug 1216291] Fix photo field definition in users api serializer. #1260

Merged
merged 1 commit into from
Oct 20, 2015

Conversation

johngian
Copy link
Contributor

No description provided.

@johngian
Copy link
Contributor Author

Verified it locally:

  • Checkout to master
  • GET 127.0.0.1:8000/api/v2/users/<user_id> with users's photo field permissions higher than the permissions of the API key used
  • traceback
  • Checkout to this PR
  • GET
  • no traceback

@johngian johngian added the DNM label Oct 20, 2015
@johngian johngian force-pushed the fix-1216291 branch 2 times, most recently from 146e619 to 0dff3d4 Compare October 20, 2015 09:51
@johngian
Copy link
Contributor Author

@akatsoulas r?

@johngian johngian added review? and removed DNM labels Oct 20, 2015
@akatsoulas
Copy link
Contributor

Reproduced this locally with the steps above. r+ 🍺

johngian added a commit that referenced this pull request Oct 20, 2015
[Fix bug 1216291] Fix photo field definition in users api serializer.
@johngian johngian merged commit f439730 into mozilla:master Oct 20, 2015
@johngian johngian deleted the fix-1216291 branch November 11, 2016 11:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants