-
-
Notifications
You must be signed in to change notification settings - Fork 587
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
📖 indexrst & readme updated for 12.0 #758
Conversation
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.
web_debranding/README.rst
Outdated
@@ -68,74 +68,27 @@ Removes references to odoo.com: | |||
24. *(feature is not required in 12.0+ versions)* | |||
25. *(feature is not required in 12.0+ versions)* |
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.
This list is duplicate for one in html and has to be deleted
* Form: https://www.it-projects.info/page/website.contactus | ||
* Facebook: https://m.me/itprojectsllc | ||
* Skype: skype@it-projects.info | ||
Tested on Odoo 12.0 288662a9de7420deaf7b13c9a8b1b1b92e15ec1f |
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 file doesn't have Maintainer section as described in the guildelines https://odoo-development.readthedocs.io/en/latest/dev/docs/README.rst.html
Check other sections too
web_debranding/README.rst
Outdated
@@ -3,7 +3,7 @@ | |||
:alt: License: LGPL-3 | |||
|
|||
==================== | |||
Backend debranding | |||
Website debranding |
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.
It's actually Backend debranding
web_debranding/doc/index.rst
Outdated
Auto-debrand new databases | ||
========================== | ||
|
||
To automatically install this module for every new databases set 'auto_install': True in **__openerp__.py** files of following modules: |
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.
openerp could be replaced to manifest (it's so since odoo 10)
<li> Replaces links to <a href="https://www.odoo.com/documentation"> documentation </a> (e.g. "Help" in Import tool, "How-to" in paypal, etc.) to custom website </li> | ||
<li> Removes official videos in planner </li> | ||
<li> Replaces "Odoo" in <i>application installed</i> mails </li> | ||
</ul> |
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.
I'd like to keep a numbered list (use<ol>
instead of <ul>
). Also, keep removed items ("the feature is not needed since ...") -- it's allows to compare feature across different versions of the module
<li> Replaces "odoo.com" in hints, examples, etc. </li> | ||
<li> Rename "OdooBot" to "Bot". Use company's logo as bot avatar </li> | ||
<li> [ENTERPRISE] Replaces icons for android and apple devices with custom url </li> | ||
<li> Replaces links to <a href="https://www.odoo.com/documentation"> documentation </a> (e.g. "Help" in Import tool, "How-to" in paypal, etc.) to custom website </li> |
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.
Second part of this message is ignored
I'd like to keep a numbered list (use<ol> instead of <ul>). Also, keep removed items ("the feature is not needed since ...") -- it's allows to compare feature across different versions of the module
web_debranding/doc/index.rst
Outdated
|
||
* Open *addons/mail/data/mail_data.xml* and edit Template **Notification Email** - delete ``using Odoo`` | ||
* Open *addons/website_livechat/data/website_livechat_data.xml* and edit in ``im_livechat_channel_data_website`` record *YourWebsiteWithOdoo.com* string | ||
* Install ``website_debranding`` module if module *Website Builder* has been already installed in your system |
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.
make a link to apps store
</p> | ||
</div> | ||
<div class="oe_row_img oe_centered"> | ||
<img class="oe_demo oe_picture oe_screenshot" src="debranding10.png"/> |
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.
Seems that the picture is missing
web_debranding/doc/index.rst
Outdated
* Install ``website_debranding`` module if module *Website Builder* has been already installed in your system | ||
* Install ``pos_debranding`` module if module `POS` has been already installed in your system | ||
* Install ``website_debranding`` `module <https://apps.odoo.com/apps/modules/10.0/website_debranding/>`__ if module *Website Builder* has been already installed in your system | ||
* Install ``pos_debranding`` `module <https://apps.odoo.com/apps/modules/10.0/pos_debranding/>`__ if module `POS` has been already installed in your system |
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 link text should describe what is referenced to. "Module" is not the best option
No description provided.