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

Add transactions support #322

Open
xiaoyi510 opened this issue Sep 21, 2020 · 5 comments
Open

Add transactions support #322

xiaoyi510 opened this issue Sep 21, 2020 · 5 comments
Labels
help wanted status:ready for adoption Feel free to implement this issue. type:enhancement Enhancement

Comments

@xiaoyi510
Copy link

MongoDB 4.x 已经支持事务,但是yii2扩展mongodb暂不支持事务操作

MongoDB 4.x already supports transactions, but yii2 extended mongodb does not support transaction operations temporarily

@bizley bizley added status:ready for adoption Feel free to implement this issue. type:enhancement Enhancement labels Sep 21, 2020
@samdark samdark changed the title yii2 MongoDB extension is old Add transactions support Sep 21, 2020
@samdark
Copy link
Member

samdark commented Sep 21, 2020

There are two pull requests:

In order to properly check it we need to switch to GitHub actions and fix tests: #279

@alxark
Copy link

alxark commented Jul 26, 2021

any plans to implement transactions?

@bizley
Copy link
Member

bizley commented Jul 26, 2021

Would you like to help with the implementation?

@alxark
Copy link

alxark commented Jul 26, 2021

Would you like to help with the implementation?

what about this PR #276 ? at least to start from.

@bizley
Copy link
Member

bizley commented Jul 27, 2021

Yes, you can continue from there. We switched to Github actions so it should be easier, the PR requires tests though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted status:ready for adoption Feel free to implement this issue. type:enhancement Enhancement
Projects
None yet
Development

No branches or pull requests

4 participants