Skip to content
This repository has been archived by the owner on Mar 28, 2019. It is now read-only.

Postgres userid fix #70

Merged
merged 7 commits into from Mar 4, 2015
Merged

Postgres userid fix #70

merged 7 commits into from Mar 4, 2015

Conversation

almet
Copy link
Contributor

@almet almet commented Mar 4, 2015

@almet almet added the to review label Mar 4, 2015
@almet almet self-assigned this Mar 4, 2015
Natim
Natim reviewed Mar 4, 2015
View reviewed changes
cliquet/tests/test_storage.py
not_updated = self.storage.get(self.resource, self.user_id,
record['id'])

self.assertNotIn("another", not_updated)
Copy link
Contributor

Choose a reason for hiding this comment

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

We don't need to add a similar test in views ?

@Natim
Copy link
Contributor

Natim commented Mar 4, 2015

r+

almet added a commit that referenced this pull request Mar 4, 2015
@almet
Merge pull request #70 from mozilla-services/postgres-userid-fix
cb63dbf 
Postgres userid fix
@almet almet merged commit cb63dbf into master Mar 4, 2015
@almet almet removed the to review label Mar 4, 2015
@almet almet deleted the postgres-userid-fix branch March 4, 2015 18:58
@leplatrem leplatrem mentioned this pull request Mar 5, 2015
Closed
glasserc pushed a commit that referenced this pull request May 20, 2016
@leplatrem
Merge pull request #71 from mozilla-services/71-handle-personal-bucket
d3dcf25 
Add personal bucket 'default' (fixes #70, fixes #69)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@almet almet

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@almet @Natim