-
-
Notifications
You must be signed in to change notification settings - Fork 667
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
[11.0][MIG] Migrate hr_experience #437
Conversation
The travis issue is unrelated with the module. I proposed a fix anyway in a seperate PR: |
hr_experience/README.rst
Outdated
@@ -0,0 +1,112 @@ | |||
.. image:: https://img.shields.io/badge/licence-AGPL--3-blue.svg |
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.
@ddufresne Thanks for the migration, can you please change to png files.
hr_experience/__manifest__.py
Outdated
|
||
{ | ||
"name": "Experience Management", | ||
"version": "1.0.0", |
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.
Update version number
<field name="certification" string="# Certificate"/> | ||
<field name="partner_id" string="Issued By"/> | ||
<field name="start_date"/> | ||
<field name="end_date"/> |
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.
Can you add expire field as well in tree view and adapt end_date domain.
<field name="certification" string="# Certificate"/> | ||
<field name="partner_id" string="Issued By"/> | ||
<field name="start_date"/> | ||
<field name="end_date"/> |
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.
The same, expire field and adapt end_date domain.
Modify academic experiences form layout Modify certificate form layout Use simple header Adjusting manifest file according OCA guidelines Add readme file view layout final touch
…nd remove useless group in views
…notebook page. In the employee form, put all fields related to the resume into the same page. 3 pages is overkill and inconsistent.
e70f6ea
to
0cabd3a
Compare
@feketemihai all fixed! |
@ddufresne Thanks, i will test on runbot when finished. |
@feketemihai before you test, I am about to add some fixes on the xml views. |
@feketemihai ok, you may check runbot. Thanks! |
Hello here, Best, |
@foutoucour each PR requires at least 2 approvals. Please make your review for getting this merged. |
[BSSFL-540] Sale / Quotation contact
No description provided.