-
Notifications
You must be signed in to change notification settings - Fork 97
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
Transaction support proposal #741
Comments
Is there a strong motivation to use |
No, .$transaction is inspired by Prismajs's API, it's a clear way to highlight the section like an atomic operation (a different kind of section, a special one) |
@gsi-alejandro .$transactions will it be made available as an “interactive” transaction? |
Yea I think we should us $.transaction @ejscribner . Also are we planning to make Ottoman available from edge environments ? |
Ref: CBSE-14946
Proposal
Steps:
Note: the way to support transactions will be by passing the ctx via options to the existing method.
Affected methods:
Ottoman:
Model:
Document:
Notice: Bring support to refdoc indexes will be a little bit more complex, no API change is required it will be done under the hood, but you need to be aware of the work to do in order to achieve transactions if refdoc logic is triggered.
The text was updated successfully, but these errors were encountered: