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.

Sign up for GitHub

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

Jump to bottom

Update validation.rst #6085

Closed
wants to merge 1 commit into from
Closed

Update validation.rst #6085

wants to merge 1 commit into from

Conversation

syedi
Copy link
Contributor

@syedi syedi commented Dec 29, 2015

at line 647 put 'IsTrue' instead of 'True'

@syedi
Update validation.rst
5c601cb 
at line 647 put 'IsTrue' instead of 'True'
@syedi
Copy link
Contributor Author

syedi commented Dec 29, 2015

Q A
Doc fix? yes
New docs? no
Applies to 3.0
Fixed tickets N/A

@xabbuh
Copy link
Member

xabbuh commented Dec 29, 2015

👍

xabbuh added a commit that referenced this pull request Dec 31, 2015
@xabbuh
minor #6085 Update validation.rst (syedi)
9b6fbd9 
This PR was submitted for the 3.0 branch but it was merged into the 2.3 branch instead (closes #6085).

Discussion
----------

Update validation.rst

at line 647 put 'IsTrue' instead of 'True'

Commits
-------

e9cfc09 Update validation.rst
xabbuh added a commit that referenced this pull request Dec 31, 2015
@xabbuh
[#6085] update XML config too
cefd184
@xabbuh
Copy link
Member

xabbuh commented Dec 31, 2015

Thank you @syedi. I have merged your PR into the 2.3 branch (that's why it is shown as closed instead of merged) and also updated the XML format in cefd184.

@xabbuh xabbuh closed this Dec 31, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Status: Reviewed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@syedi @xabbuh