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

Implement pointer types in postgres. #2086

Merged
merged 3 commits into from
Jun 17, 2016

Conversation

drew-gross
Copy link
Contributor

Store pointers as strings because in Parse you can create pointers to objects that don't exist (and even classes that don't exist)

Based on #2081, merge that first then update this branch on top of that.

@ghost
Copy link

ghost commented Jun 17, 2016

@drew-gross updated the pull request.

@flovilmart
Copy link
Contributor

Just merged, you still have conflicts, maybe because of the squash merge

@drew-gross
Copy link
Contributor Author

Probably yea. Will rebase.

@ghost
Copy link

ghost commented Jun 17, 2016

@drew-gross updated the pull request.

@flovilmart
Copy link
Contributor

Waiting for travis but LGTM

@codecov-io
Copy link

codecov-io commented Jun 17, 2016

Current coverage is 89.75%

Merging #2086 into master will decrease coverage by 0.03%

@@             master      #2086   diff @@
==========================================
  Files            93         93          
  Lines          6764       6767     +3   
  Methods        1191       1190     -1   
  Messages          0          0          
  Branches       1430       1431     +1   
==========================================
  Hits           6074       6074          
- Misses          690        693     +3   
  Partials          0          0          

Powered by Codecov. Last updated by ab06055...77e5775

@ghost
Copy link

ghost commented Jun 17, 2016

@drew-gross updated the pull request.

@drew-gross drew-gross merged commit 006540c into parse-community:master Jun 17, 2016
@drew-gross drew-gross deleted the pg-next branch June 17, 2016 18:09
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.

3 participants