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

[17.0][MIG] fieldservice_repair: Migration to 17.0 #1209

Merged
merged 38 commits into from
Sep 26, 2024

Conversation

imlopes
Copy link

@imlopes imlopes commented Sep 9, 2024

Depends on #1205

Sandip Mangukiya and others added 30 commits September 9, 2024 10:16
* [IMP] fieldservice_repair
Currently translated at 100.0% (10 of 10 strings)

Translation: field-service-14.0/field-service-14.0-fieldservice_repair
Translate-URL: https://translation.odoo-community.org/projects/field-service-14-0/field-service-14-0-fieldservice_repair/it/
Currently translated at 100.0% (10 of 10 strings)

Translation: field-service-14.0/field-service-14.0-fieldservice_repair
Translate-URL: https://translation.odoo-community.org/projects/field-service-14-0/field-service-14-0-fieldservice_repair/es_AR/
Currently translated at 100.0% (8 of 8 strings)

Translation: field-service-14.0/field-service-14.0-fieldservice_repair
Translate-URL: https://translation.odoo-community.org/projects/field-service-14-0/field-service-14-0-fieldservice_repair/pt_BR/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: field-service-15.0/field-service-15.0-fieldservice_repair
Translate-URL: https://translation.odoo-community.org/projects/field-service-15-0/field-service-15-0-fieldservice_repair/
Currently translated at 100.0% (6 of 6 strings)

Translation: field-service-15.0/field-service-15.0-fieldservice_repair
Translate-URL: https://translation.odoo-community.org/projects/field-service-15-0/field-service-15-0-fieldservice_repair/de/
imlopes pushed a commit to yankinmax/field-service that referenced this pull request Sep 11, 2024
While migrating ``fieldservice_repair`` (OCA#1209) some unit tests were failing because this module adds ``required`` condition on two new fields on ``fms.equipment`` model. 
To avoid this, the best solution was to migrate the required to the view as we cannot force the installation order on odoo.
@max3903 max3903 added this to the 17.0 milestone Sep 19, 2024
@max3903 max3903 self-assigned this Sep 19, 2024
@max3903 max3903 closed this Sep 21, 2024
@max3903 max3903 reopened this Sep 21, 2024
@max3903 max3903 closed this Sep 23, 2024
@max3903 max3903 reopened this Sep 23, 2024
@max3903
Copy link
Member

max3903 commented Sep 23, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 17.0-ocabot-merge-pr-1209-by-max3903-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Sep 23, 2024
Signed-off-by max3903
@OCA-git-bot
Copy link
Contributor

@max3903 your merge command was aborted due to failed check(s), which you can inspect on this commit of 17.0-ocabot-merge-pr-1209-by-max3903-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@imlopes
Copy link
Author

imlopes commented Sep 26, 2024

@max3903 can you retry, please ?
Regarding the logs, I don't know whats is the problem, it seems just a connection issue.

Huge thanks

@max3903
Copy link
Member

max3903 commented Sep 26, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 17.0-ocabot-merge-pr-1209-by-max3903-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit efdaf46 into OCA:17.0 Sep 26, 2024
7 of 11 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at bcb56b7. Thanks a lot for contributing to OCA. ❤️

EdgarRetes pushed a commit to ursais/field-service that referenced this pull request Oct 16, 2024
While migrating ``fieldservice_repair`` (OCA#1209) some unit tests were failing because this module adds ``required`` condition on two new fields on ``fms.equipment`` model.
To avoid this, the best solution was to migrate the required to the view as we cannot force the installation order on odoo.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.