Skip to content

PHPLIB-468: Add documentation for with_transaction #680

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

Merged
merged 1 commit into from
Sep 9, 2019

Conversation

alcaeus
Copy link
Member

@alcaeus alcaeus commented Sep 5, 2019

https://jira.mongodb.org/browse/PHPLIB-468

I ended up adding a functions chapter after all, as I didn't want to just put the function somewhere. The tutorial is copied from the documentation examples, I feel that is most familiar. We can always add more examples if we feel that's necessary.

@alcaeus alcaeus requested a review from jmikola September 5, 2019 16:33
--------

- :php:`MongoDB\\Driver\\Session::startTransaction <mongodb-driver-session.starttransaction>`
- :php:`MongoDB\\Driver\\Session::commitTransaction <mongodb-driver-session.committransaction>`
Copy link
Member

Choose a reason for hiding this comment

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

Let's add:

- :manual:`Transactions: Drivers API </core/transactions-in-applications>` documentation in the MongoDB manual
- `Convenient API for Transactions <https://github.com/mongodb/specifications/blob/master/source/transactions-convenient-api/transactions-convenient-api.rst>`_ specification

Based on other docs pages, this can go after the previous PHPC links.

Also, the manual link here pertains to my later suggestion to remove the tutorial page introduced in this PR.

@alcaeus alcaeus force-pushed the phplib-468-docs branch 2 times, most recently from 5da9567 to 9cda389 Compare September 9, 2019 12:29
alcaeus added a commit that referenced this pull request Sep 9, 2019
@alcaeus alcaeus merged commit 7e0337c into mongodb:master Sep 9, 2019
@alcaeus alcaeus deleted the phplib-468-docs branch September 9, 2019 13:15
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.

2 participants