[13.0][MIG] fieldservice_account_analytic #773
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- This change removes most of the invoicing logic from fieldservice_account and puts it into its own module
…invoice* and fsm_order
Currently translated at 11.1% (2 of 18 strings) Translation: field-service-12.0/field-service-12.0-fieldservice_account_analytic Translate-URL: https://translation.odoo-community.org/projects/field-service-12-0/field-service-12-0-fieldservice_account_analytic/es_CL/
[IMP] Move Method Up [IMP] api.model + space [IMP] fsm_parent_id [FIX] Typo
Currently translated at 100.0% (18 of 18 strings) Translation: field-service-12.0/field-service-12.0-fieldservice_account_analytic Translate-URL: https://translation.odoo-community.org/projects/field-service-12-0/field-service-12-0-fieldservice_account_analytic/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: field-service-12.0/field-service-12.0-fieldservice_account_analytic Translate-URL: https://translation.odoo-community.org/projects/field-service-12-0/field-service-12-0-fieldservice_account_analytic/
Currently translated at 100.0% (17 of 17 strings) Translation: field-service-12.0/field-service-12.0-fieldservice_account_analytic Translate-URL: https://translation.odoo-community.org/projects/field-service-12-0/field-service-12-0-fieldservice_account_analytic/pt_BR/
[IMP] Remove Unused Import [IMP] Github Comments [IMP] Add FSM to Variable
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: field-service-12.0/field-service-12.0-fieldservice_account_analytic Translate-URL: https://translation.odoo-community.org/projects/field-service-12-0/field-service-12-0-fieldservice_account_analytic/
Currently translated at 100.0% (21 of 21 strings) Translation: field-service-12.0/field-service-12.0-fieldservice_account_analytic Translate-URL: https://translation.odoo-community.org/projects/field-service-12-0/field-service-12-0-fieldservice_account_analytic/pt_BR/
Currently translated at 100.0% (21 of 21 strings) Translation: field-service-12.0/field-service-12.0-fieldservice_account_analytic Translate-URL: https://translation.odoo-community.org/projects/field-service-12-0/field-service-12-0-fieldservice_account_analytic/pt_BR/
Currently translated at 100.0% (21 of 21 strings) Translation: field-service-12.0/field-service-12.0-fieldservice_account_analytic Translate-URL: https://translation.odoo-community.org/projects/field-service-12-0/field-service-12-0-fieldservice_account_analytic/es_AR/
Currently translated at 9.5% (2 of 21 strings) Translation: field-service-12.0/field-service-12.0-fieldservice_account_analytic Translate-URL: https://translation.odoo-community.org/projects/field-service-12-0/field-service-12-0-fieldservice_account_analytic/pt_PT/
Currently translated at 100.0% (21 of 21 strings) Translation: field-service-12.0/field-service-12.0-fieldservice_account_analytic Translate-URL: https://translation.odoo-community.org/projects/field-service-12-0/field-service-12-0-fieldservice_account_analytic/es_AR/
This was referenced Apr 29, 2021
baimont
force-pushed
the
13.0-mig-fieldservice_account_analytic
branch
3 times, most recently
from
c8d56b4
to
b5f8dbb
Compare
baimont
commented
May 7, 2021
.travis.yml
Outdated
| @@ -24,6 +23,9 @@ jobs: | |||
| - stage: linting | |||
| name: "pre-commit" | |||
| before_install: | |||
| - sudo add-apt-repository -y "deb https://apt-archive.postgresql.org/pub/repos/apt trusty-pgdg-archive main" | |||
There was a problem hiding this comment.
about this : postgis doesn't seem to be installable on trusty anymore.
https://www.postgresql.org/message-id/YBMtd6nRuXyU2zS4%40msg.df7cb.de
|
Hi @moylop260, my colleage @baimont is are trying to make this repo work on runbot. The problem is that the vauxoo/odoo-80-image-shippable-auto that is used on runbot is based on Ubuntu 14 trusty and postgis is not easily available for that old OS anymore. Do you know if we can use another docker image on runbot, ideally based on an ubuntu version that is available on travis too ? |
|
Hi @moylop260 thanks for the explanation. It is probably not worth putting effort into this now, as we still plan to move to GitHub actions and the new runbot. |
baimont
force-pushed
the
13.0-mig-fieldservice_account_analytic
branch
from
cae2374
to
2eb1906
Compare
baimont
force-pushed
the
13.0-mig-fieldservice_account_analytic
branch
from
2eb1906
to
b5a4e9e
Compare
… a timesheet line in a method to inherit it
baimont
force-pushed
the
13.0-mig-fieldservice_account_analytic
branch
from
dd05002
to
1a2a9fb
Compare
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
github-actions
bot
added
the
stale
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#354
Hi there,
I know @murtuzasaleh already did a fine job there #650.
But I was trying to migrate fieldservice_isp_account which depends on fieldservice_account_analytic... And I had a problem with unit tests.
Tests on #650 are not working because they depend on fieldservice_account's setUp() which creates test_location without customer_id. And since customer_id becomes required with fieldservice_account_analytic, the test doesn't go further. I propose here to solve this by removing the dependency on fieldservice_account for fieldservice_account_analytic tests.