documentation icon indicating copy to clipboard operation
documentation copied to clipboard

[IMP] CRM: updating flow and screenshots about lead mining

Open StraubCreative opened this issue 3 years ago β€’ 8 comments

Task: 2543815 Cherry pick from #998

Omitted changes proposed in the track_leads directory (e.g. lead_scoring.rst + image folder). Both the content and image updates were already pushed from another PR so what was proposed on #998 was outdated and no longer necessary.

What’s left over are the original proposed changes in #998, within the acquire_leads directory which include updates to lead_mining.rst and corresponding image files.

On top of these proposals, I went ahead and made the following changes:

  • renamed the image files in the acquire_leads asset directory, since they were using outdated naming convention (e.g. LM1.png, LM6.png, LM7.png)
  • cleaned up RST formatting issues (headings, line breaks, lists, broken links, guilabels, etc.)
  • rewrote select content to be more concise and follow content guidelines. You can see the first version of proposals here to compare the changes made.

StraubCreative avatar Jul 31 '22 22:07 StraubCreative

Hi @meng-odoo

For content review please do the following:

  • confirm that I captured the essence of what Tori was originally proposing here for lead_mining.rst
  • ensure the content (copy + screenshots) clearly and accurately illustrates the features for lead mining
  • double check XPL review on #998 to see if I hit most/all of what they suggested be changed
  • let me know if we need to change or add anything (this cherry-pick is from a PR that's 14-months old...good chance something is missing or some content is out-of-date).

Thank you πŸ™

cc: @mivu-odoo

StraubCreative avatar Jul 31 '22 22:07 StraubCreative

Nudge @meng-odoo cc: @mivu-odoo

StraubCreative avatar Sep 23 '22 20:09 StraubCreative

Hi @meng-odoo πŸ‘‹ I made your changes to the document and updated the image. I also added a :guilabel: to line 22:

After the :guilabel:Lead Mining setting is activated...

Thanks for the content updates! Let me know if you have more changes :)

samueljlieber avatar Sep 28 '22 14:09 samueljlieber

Hi @meng-odoo! Your changes have been made πŸ‘

samueljlieber avatar Sep 29 '22 18:09 samueljlieber

All set @meng-odoo πŸ™‚ Thanks!

samueljlieber avatar Sep 30 '22 14:09 samueljlieber

Hi @mivu-odoo! I implemented your changes, ready for another look! Thank you! πŸ™‚

samueljlieber avatar Oct 04 '22 19:10 samueljlieber

Thank you @mivu-odoo! Hi @StraubCreative πŸ‘‹ this doc is ready for your review!

samueljlieber avatar Oct 07 '22 13:10 samueljlieber

4af536d addresses merge conflicts from #2983 and outdated 14.0 content.

Also includes fixes to change requests above.

StraubCreative avatar Dec 05 '22 22:12 StraubCreative

@odoo/doc-review you're up πŸ™‚ I think this is the last PR from the VPK pipeline. Thanks!

StraubCreative avatar Dec 05 '22 22:12 StraubCreative