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

[MIG] hr_timesheet_sheet: Migration to 15.0 #484

Closed
wants to merge 166 commits into from

Conversation

dochse
Copy link

@dochse dochse commented Apr 22, 2022

No description provided.

MiquelRForgeFlow and others added 30 commits April 22, 2022 13:15
* [10.0] hr_timesheet_sheet

* [11.0][MIG] hr_timesheet_sheet

* [REMOVE] hr_timesheet.sheet.account

* [REMOVE] 'new' state

* [ADD] Tests

* [UPD] Adapt to multicompany

* [ADD] Add more tests (include multicompany tests)

* [FIX] project_task_stage_allow_timesheet: show error message only if task

* [ADD] Migration scripts to v11
Currently translated at 98.9% (88 of 89 strings)

Translation: hr-timesheet-11.0/hr-timesheet-11.0-hr_timesheet_sheet
Translate-URL: https://translation.odoo-community.org/projects/hr-timesheet-11-0/hr-timesheet-11-0-hr_timesheet_sheet/ja/
Currently translated at 100,0% (89 of 89 strings)

Translation: hr-timesheet-11.0/hr-timesheet-11.0-hr_timesheet_sheet
Translate-URL: https://translation.odoo-community.org/projects/hr-timesheet-11-0/hr-timesheet-11-0-hr_timesheet_sheet/pt_BR/
…ay into this module, which adds a configuration to select the week start day.
Updated by Update PO files to match POT (msgmerge) hook in Weblate.
If you run tests on Sunday, test_4 was not prepared for it, as next day is Monday,
which belongs to other week, and thus not included in same timesheet. With this,
we cover that case, decreasing one day instead of adding it.
Updated by Update PO files to match POT (msgmerge) hook in Weblate.
simonstromb and others added 16 commits April 22, 2022 13:15
Currently translated at 61.5% (112 of 182 strings)

Translation: timesheet-14.0/timesheet-14.0-hr_timesheet_sheet
Translate-URL: https://translation.odoo-community.org/projects/timesheet-14-0/timesheet-14-0-hr_timesheet_sheet/sv/
Currently translated at 88.4% (161 of 182 strings)

Translation: timesheet-14.0/timesheet-14.0-hr_timesheet_sheet
Translate-URL: https://translation.odoo-community.org/projects/timesheet-14-0/timesheet-14-0-hr_timesheet_sheet/sv/
Currently translated at 88.4% (161 of 182 strings)

Translation: timesheet-14.0/timesheet-14.0-hr_timesheet_sheet
Translate-URL: https://translation.odoo-community.org/projects/timesheet-14-0/timesheet-14-0-hr_timesheet_sheet/sv/
Currently translated at 94.5% (172 of 182 strings)

Translation: timesheet-14.0/timesheet-14.0-hr_timesheet_sheet
Translate-URL: https://translation.odoo-community.org/projects/timesheet-14-0/timesheet-14-0-hr_timesheet_sheet/sv/
Currently translated at 97.2% (177 of 182 strings)

Translation: timesheet-14.0/timesheet-14.0-hr_timesheet_sheet
Translate-URL: https://translation.odoo-community.org/projects/timesheet-14-0/timesheet-14-0-hr_timesheet_sheet/sv/
Currently translated at 98.3% (179 of 182 strings)

Translation: timesheet-14.0/timesheet-14.0-hr_timesheet_sheet
Translate-URL: https://translation.odoo-community.org/projects/timesheet-14-0/timesheet-14-0-hr_timesheet_sheet/nl/
Currently translated at 98.3% (179 of 182 strings)

Translation: timesheet-14.0/timesheet-14.0-hr_timesheet_sheet
Translate-URL: https://translation.odoo-community.org/projects/timesheet-14-0/timesheet-14-0-hr_timesheet_sheet/nl/
…voice_id field of account.analytic.line adds it).

Tests currently fail without having sale_timesheet addon installed.
TT35948
Copy link
Contributor

@dsolanki-initos dsolanki-initos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functionally test LGTM!

@dochse Can you please check failed checks?

@fuentes73
Copy link
Member

Hi @dochse can you fix the pre-commit error?

Copy link
Member

@gaikaz gaikaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mig. changes LGTM
Please check pre-commit

Comment on lines +31 to +32
'/hr_timesheet_sheet/static/src/js/backend.js',
'/hr_timesheet_sheet/static/src/scss/backend.scss'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might be an issue after all.

Suggested change
'/hr_timesheet_sheet/static/src/js/backend.js',
'/hr_timesheet_sheet/static/src/scss/backend.scss'
'hr_timesheet_sheet/static/src/js/backend.js',
'hr_timesheet_sheet/static/src/scss/backend.scss'

@JuanyDForgeflow
Copy link

@dochse are you going to continue with this?

@dsolanki-initos
Copy link
Contributor

@dochse can you please fix the checked failed?

@dsolanki-initos
Copy link
Contributor

@dochse can you please run pre-commit run -a command for fixing the GitHub checks

@dsolanki-initos
Copy link
Contributor

@astirpe can you please review this PR?

@CRogos
Copy link
Contributor

CRogos commented Aug 23, 2022

@dochse are you still working on this PR?
I've created a fork of this PR and implemented the feedback of @gaikaz and tried to fix all pre-commit checks.
But there are several warnings where the string format variables are not used in the translation. I assume the translations need to be adapted, but I am not capable of these languages.

hr_timesheet_sheet/i18n/id.po:439: [W7968(po-msgstr-variables), ] Translation string couldn't be parsed correctly using string%variables TypeError('not all arguments converted during string formatting')

What is the procedure to continue with the translation issue and if @dochse is not active anymore? Shall I create a second PR #516 ?

@dochse
Copy link
Author

dochse commented Aug 23, 2022

@dochse are you still working on this PR? I've created a fork of this PR and implemented the feedback of @gaikaz and tried to fix all pre-commit checks. But there are several warnings where the string format variables are not used in the translation. I assume the translations need to be adapted, but I am not capable of these languages.

hr_timesheet_sheet/i18n/id.po:439: [W7968(po-msgstr-variables), ] Translation string couldn't be parsed correctly using string%variables TypeError('not all arguments converted during string formatting')

What is the procedure to continue with the translation issue and if @dochse is not active anymore? Shall I create a second PR #516 ?

Im not actively working on it anymore, so you are free to do with this as you will

@pedrobaeza
Copy link
Member

Supersede by #516

@pedrobaeza pedrobaeza closed this Sep 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.