-
-
Notifications
You must be signed in to change notification settings - Fork 672
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
[18.0][MIG] hr_employee_firstname: Migration to 18.0 #1397
Merged
OCA-git-bot
merged 42 commits into
OCA:18.0
from
vauxoo-dev:18.0-mig-hr-employee-firstname
Oct 31, 2024
Merged
Changes from 1 commit
Commits
Show all changes
42 commits
Select commit
Hold shift + click to select a range
39faa49
[ADD] hr_employee_firstname
feketemihai 4e55fbd
Remove field name related, change init script, add unittests
5646b75
[FIX+IMP] hr_employee_firstname: Several things:
adrienpeiffer a3421ed
[IMP][hr_employee_firstname] Avoid to update partners without reason
adrienpeiffer bd7fbc1
[FIX] Update tests in hr_employee_firstname - failing travis
f83df0d
[REF] hr_employee_firstname: Split names at module install to get the…
feketemihai b2e60ec
[MIG] hr_employee_firstname: Migrated to 10.0 (#276)
espo-tony 257469a
[MIG] hr_employee_firstname: Migration to version 11.0
892ec93
[FIX] employee_firstname: test (#427)
feketemihai e542fe9
[11.0][FIX] Fix dangerous use of 'replace' from hr.employee view.
ddufresne fdcc0e9
[12.0][MIG] hr_employee_firstname
astirpe b08e054
[FIX/IMP] Make required attributes dynamic
astirpe a37d32b
[13.0][MIG] hr_employee_firstname
astirpe 6080f1b
pre-commit
astirpe e5b43ba
pre-commit update
OCA-git-bot a70ba24
[13.0][FIX] hr_employee_firstname : testscript
Saran440 f070a6f
[IMP] hr_employee_firstname: ease extension
astirpe 20f6385
hr_employee_firstname 13.0.1.0.1
OCA-git-bot 4bcfc4f
[MIG] hr_employee_firstname: Migration to 14.0
luistorresm aba958f
[REF] hr_employee_firstname: Removed partner_firstname dependency
eaca361
[IMP] update dotfiles [ci skip]
OCA-git-bot b2f54d2
hr_employee_firstname 14.0.2.0.0
OCA-git-bot 28f555e
[FIX] hr_employee_firstname: Avoid access error on employee update
luistorresm 67b9411
hr_employee_firstname 14.0.2.0.1
OCA-git-bot 7f099e8
[FIX] hr_employee_firstname: Move fields to common employee
hugho-ad c5cceec
hr_employee_firstname 14.0.3.0.0
OCA-git-bot dd32419
[MIG] hr_employee_firstname: Migration to 15.0
luistorresm 139ea4a
[MIG] hr_employee_firstname: Migration to 16.0
luistorresm 6186d56
[UPD] README.rst
OCA-git-bot a453dd1
[FIX] hr_employee_firstname: Improved create on employee
luistorresm 7e70006
[16.0][FIX] test refresh->invalidate_recordset
FernandoRomera 426ef60
hr_employee_firstname 16.0.1.0.1
OCA-git-bot 5ca3a01
[UPD] README.rst
OCA-git-bot 9f766dd
[IMP] hr_employee_firstname: rename hooks file
andreagidaltig 3102e50
[BOT] post-merge updates
OCA-git-bot 06dfa95
[IMP] hr_employee_firstname: pre-commit stuff
andreagidaltig 1d311d1
[MIG] hr_employee_firstname: Migration to 17.0
andreagidaltig ef66013
[BOT] post-merge updates
OCA-git-bot a6a9d5a
[FIX] hr_employee_firstname: fix DeprecationWarning
edy1192 c5b7bd7
[BOT] post-merge updates
OCA-git-bot a4b8da6
[IMP] hr_employee_firstname: pre-commit auto fixes
edy1192 7576692
[MIG] hr_employee_firstname: Migration to 18.0
edy1192 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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 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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why this change?
I couldn't find any related change that doesn't allow the previous syntax, and I think importing methods directly from
odoo.tools
is preferred.If really required, it might be a good idea to document it in the commit description.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When I ran the tests to verify that they worked fine I got this error:
I tried to import the tools library using
from odoo import tools
and thentools.submap
but got the same error.