[ADD] Referrals: new doc explaining all of referrals #6206
Conversation
larm-odoo
force-pushed
the
15.0-Referrals-new-how-to-in-depth-doc-larm
branch
from
28d34bd to
cd9e30f
Compare
jero-odoo
left a comment
jero-odoo
left a comment
There was a problem hiding this comment.
hey @larm-odoo nice work. I had some suggestions, mostly on formatting and organization. Let me know if you have any questions. Thanks
larm-odoo
force-pushed
the
15.0-Referrals-new-how-to-in-depth-doc-larm
branch
from
cd9e30f to
0593649
Compare
|
Thanks @jero-odoo! All comments either accepted/added to the doc or explained (not many I didn't take). I tagged you again for another look and an approval. Thanks in advance! |
jero-odoo
left a comment
jero-odoo
left a comment
There was a problem hiding this comment.
hey @larm-odoo just a few changes (mostly grammar and spelling). Good to go otherwise 👍
larm-odoo
force-pushed
the
15.0-Referrals-new-how-to-in-depth-doc-larm
branch
from
0593649 to
7b9b4c3
Compare
|
Hi @tiku-odoo - this is ready for you! There is a doc link in here that links to a recruitment doc that is NOT published yet- so that error is visible, be aware. It will not appear once that is published. Also you can disregard any errors from 'code owner' per Sam. |
tiku-odoo
left a comment
tiku-odoo
left a comment
There was a problem hiding this comment.
Awesome job on this doc, really great flows and explanations.
I still have some more to go on this doc and I didn't want to rush through it as I've left a few functional edit suggestions above that I don't want to miss any below. Hold off on making any edits until I can finish out the doc.
Thanks and great work on this doc! Have a nice weekend
tiku-odoo
left a comment
tiku-odoo
left a comment
There was a problem hiding this comment.
Nice work on the document. I made a few comments and it's ready for your review.
I noticed that you didn't cover friends in this document, configuration--> Friends
I would also like to know the level of access needed to make the various edits you review in the doc (as it is mixed throughout the doc)
Looking forward to seeing this published!
|
Thank you @tiku-odoo! I incorporated your edits - thanks for finding some things I didn't catch! All set for you, @odoo/us-doc-review! Please note, the error for the document 'recruitment' is currently on #5994 - this referrals document must be published after the recruitment doc, so the link works. |
larm-odoo
force-pushed
the
15.0-Referrals-new-how-to-in-depth-doc-larm
branch
from
7b9b4c3 to
32abdc0
Compare
larm-odoo
force-pushed
the
15.0-Referrals-new-how-to-in-depth-doc-larm
branch
from
32abdc0 to
cbee644
Compare
| - :guilabel:`Contract Signed`: 50 points | ||
|
|
||
| The total points earned for a referral that is hired is 85 points. Modifications of the points | ||
| earned for each stage is done in the Recruitment application. Refer to the :doc:`../recruitment` |
There was a problem hiding this comment.
Hi @larm-odoo, I am commenting on this line (552) regarding the ../recruitment build error, can you please fix this before I review?
Thank you!
There was a problem hiding this comment.
All fixed @samueljlieber! The link currently sends you to a doc that is being updated on #5994. So when you click it, you see this page: https://www.odoo.com/documentation/17.0/applications/hr/recruitment.html
Currently, this is mostly blank, and just has a link to the Job Positions doc. But PR #5994 is the detailed Hiring Flow doc, which is what this will be pointing to, ultimately.
larm-odoo
force-pushed
the
15.0-Referrals-new-how-to-in-depth-doc-larm
branch
from
cbee644 to
0ae0ade
Compare
samueljlieber
force-pushed
the
15.0-Referrals-new-how-to-in-depth-doc-larm
branch
from
0ae0ade to
4e469a2
Compare
samueljlieber
left a comment
samueljlieber
left a comment
There was a problem hiding this comment.
Nice job @larm-odoo! This PR looks good to me. I just pushed up a commit updating the :ref: on line 552 to point to the correct section in recruitment.rst that will be added in #5994.
@StraubCreative this PR is good to go, but needs to be merged AFTER #5994 👍
|
@samueljlieber #5994 has been merged along with it's forward-port chain, just now, on #7099 Can merge here after rebase and CI checks pass, thanks! @robodoo delegate=samueljlieber |
samueljlieber
force-pushed
the
15.0-Referrals-new-how-to-in-depth-doc-larm
branch
from
4e469a2 to
8ba5f1a
Compare
|
Rebased to fix CI check error, I will push up one more commit to fix a few items in the |
samueljlieber
force-pushed
the
15.0-Referrals-new-how-to-in-depth-doc-larm
branch
from
8ba5f1a to
09d3e39
Compare
closes #6206 Signed-off-by: Samuel Lieber (sali) <[email protected]>
7 participants
New document explaining all of Referrals. Will be forward ported to 16, then 17 when that is live, and targeted edits will be made for those versions.