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

fix #24780 change default isolation method in dol_clone #29391

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

Al-thi
Copy link
Contributor

@Al-thi Al-thi commented Apr 18, 2024

No description provided.

@Al-thi Al-thi marked this pull request as draft April 18, 2024 12:54
@eldy eldy added the PR to fix - See feedback in comments PR needs to be fixed to be integrated (some comments should describes the fix to do) label Apr 18, 2024
@Al-thi Al-thi changed the base branch from 17.0 to develop April 18, 2024 13:22
@Al-thi Al-thi force-pushed the fix/24780 branch 2 times, most recently from 9621378 to dd50db3 Compare April 18, 2024 13:53
@Al-thi
Copy link
Contributor Author

Al-thi commented Apr 18, 2024

I am having Travis CI issues, I see a lot of failure from ppa.launchpad.net for multiple develop PRs.

Shouldn't we implement apt retry mechanism in travis configuration ?

cf. travis-ci/travis-ci#9112 (comment)

@Al-thi Al-thi marked this pull request as ready for review April 18, 2024 14:45
@Al-thi Al-thi force-pushed the fix/24780 branch 2 times, most recently from 31eeeb2 to e484a58 Compare April 19, 2024 07:46
@Al-thi Al-thi requested a review from eldy April 19, 2024 12:43
This fixes serialization error when attempting to update a custom field value.
This change is made globally because the previous native = 0 default isolation
method is labeled as `deprecated` in the code and therefore shouldn't be used
anymore.

Signed-off-by: Alexis Thietard <alexis.thietard@rtone.fr>
@eldy eldy added PR postponed PR is postponed (will be processed later). Ie: feature pushed during a beta or need transition steps and removed PR to fix - See feedback in comments PR needs to be fixed to be integrated (some comments should describes the fix to do) labels Apr 22, 2024
@eldy eldy merged commit 8c9a1c2 into Dolibarr:develop Nov 13, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR postponed PR is postponed (will be processed later). Ie: feature pushed during a beta or need transition steps
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants