Skip to content

please correct the php Relation implementation doc at http://docs.parseplatform.org/php/guide/#relations #321

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

Closed
akshaychauhan7737 opened this issue Jun 9, 2017 · 4 comments

Comments

@akshaychauhan7737
Copy link

hello,
please correct parse php guide for Relation at : http://docs.parseplatform.org/php/guide/#relations
the code there is the js code not the php code..
I could not find a contact form on http://docs.parseplatform.org/ so i am adding this as a issue over here

@montymxb
Copy link
Contributor

montymxb commented Jun 9, 2017

@akshaychauhan7737 I took a look at the docs, it seems that's not the only section that seems to be JS code instead. Parts of the Security section are incorrect as well. Thanks for bringing this to our attention, we'll look into where the old references have gone and we'll get those back if we can.

::EDIT::
It appears the references internally are correct. However the JS code showing by default appears to be a bug regarding which SDK examples should be displayed for. Looking at other sdk guides appears to show the same issue in the other guides.

Given this is not an issue with the php sdk itself I'll move this over to the docs project. Thanks for bringing this up by the way, hopefully we'll have it resolved soon.

@montymxb
Copy link
Contributor

montymxb commented Jun 9, 2017

This issue has been moved over to the relevant doc mentioned above. You can keep an eye out there for any progress that's made.

@montymxb montymxb closed this as completed Jun 9, 2017
@montymxb
Copy link
Contributor

montymxb commented Jun 9, 2017

@akshaychauhan7737 just notifying you that I was able to find the underlying issue and have since pushed up a PR to docs. Once that's approved this will correct the observed issue not just in the php sdk but in all others as well.

@montymxb
Copy link
Contributor

montymxb commented Jun 9, 2017

A patch has been merged in for this issue. Docs look good now 👍 . Thanks for bringing this to our attention!

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

No branches or pull requests

2 participants