Skip to content
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

[IMP] general: iot windows iot update process #8361

Closed
wants to merge 1 commit into from

Conversation

@robodoo
Copy link
Collaborator

robodoo commented Mar 25, 2024

@C3POdoo C3POdoo requested a review from a team March 25, 2024 13:50
@tiku-odoo
Copy link
Contributor Author

@lse-odoo
@brse-odoo

Can you each look at this PR when you have a moment? There should only be minor changes as this copy has been rearranged for re-installation from the Windows installation page, which was recently updated.

Thanks in advance for your help on this doc!

Tim 👍

Copy link
Contributor

@brse-odoo brse-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tiku-odoo I've finished my review of this PR. Great job! I only had a few suggestions for you to accept/reject as you see fit - I'm approving this PR for the next round.

Let me know if you have any questions about my comments. Thanks!

content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/windows_iot.rst Outdated Show resolved Hide resolved
@tiku-odoo
Copy link
Contributor Author

@wigu-odoo

Can you review these changes when you have a moment?

@tiku-odoo tiku-odoo force-pushed the 16.0-general-iot-updating-windows-iot-tiku branch from 640d8d8 to 190b1e3 Compare March 25, 2024 16:30
@tiku-odoo tiku-odoo requested a review from ksc-odoo March 25, 2024 16:31
@tiku-odoo
Copy link
Contributor Author

@ksc-odoo

This PR is ready for your review when you have a moment.

Thanks, 👍
Tim

@tiku-odoo tiku-odoo force-pushed the 16.0-general-iot-updating-windows-iot-tiku branch 2 times, most recently from 0e610d8 to b094039 Compare March 25, 2024 19:05
@tiku-odoo
Copy link
Contributor Author

@ksc-odoo
Pushed up a small change on this PR (defining the link for download better).

This is a mashup of the Windows IOT doc. The focus here is upgrading (not installation).

Ready for your review when you have some time.

Thanks,
Tim

Copy link

@wigu-odoo wigu-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good on my end!

Copy link
Contributor

@ksc-odoo ksc-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hey @tiku-odoo -- just finished my Final Review of this PR. Once you implement the necessary changes, tag me again for another quick look. Thanks! 👍

content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
@tiku-odoo tiku-odoo force-pushed the 16.0-general-iot-updating-windows-iot-tiku branch from b094039 to 6ddf0d3 Compare March 26, 2024 13:47
@tiku-odoo tiku-odoo requested a review from ksc-odoo March 26, 2024 13:48
@tiku-odoo
Copy link
Contributor Author

@ksc-odoo

This doc is ready for another look!

Thanks,
Tim ,

Copy link
Contributor

@ksc-odoo ksc-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work implementing the previously-suggested changes, @tiku-odoo -- there are only a handful of minor adjustments that require your attention. But, I'm going to 'approve' now, because I don't think you'll have any issues implementing them during this round of revision. Once you think it's ready, you can tag it for Tech Review. Thanks! 👍

content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
@tiku-odoo tiku-odoo force-pushed the 16.0-general-iot-updating-windows-iot-tiku branch from 6ddf0d3 to 875fc98 Compare March 26, 2024 19:37
@tiku-odoo
Copy link
Contributor Author

@samueljlieber

This doc is ready for your review when you have a moment.

Thanks,
Tim 👍

@tiku-odoo tiku-odoo force-pushed the 16.0-general-iot-updating-windows-iot-tiku branch from 875fc98 to 44fa6ed Compare March 27, 2024 17:25
@tiku-odoo
Copy link
Contributor Author

@samueljlieber

I've added an additional admonition block to address @lse-odoo comments.

This is ready for your review now.

Thanks,
Tim

Copy link
Contributor

@samueljlieber samueljlieber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @tiku-odoo, nice job on this improvement PR. I have a few suggestions and corrections, please see below and my comment about the uninstallation notice.

Also, please update the commit message to have a space after the colon :)

Let me know your thoughts on the rephrasing, and tag me for another look once this PR is ready. Thank you!

content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
content/applications/general/iot/config/updating_iot.rst Outdated Show resolved Hide resolved
@tiku-odoo tiku-odoo force-pushed the 16.0-general-iot-updating-windows-iot-tiku branch from 44fa6ed to 758aeca Compare March 28, 2024 17:39
@tiku-odoo tiku-odoo changed the title [IMP] general:iot windows iot update process [IMP] general: iot windows iot update process Mar 28, 2024
@tiku-odoo tiku-odoo force-pushed the 16.0-general-iot-updating-windows-iot-tiku branch from 758aeca to e3f85e6 Compare March 28, 2024 17:56
@tiku-odoo
Copy link
Contributor Author

@samueljlieber

I've made the requested changes; thanks for your help streamlining it.

Ready for a final review.

Thanks,
Tim 👍

Copy link
Contributor

@samueljlieber samueljlieber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice @tiku-odoo, thanks for implementing the suggestions from my previous review! I have just a couple more quick fixes that I will push up after this review to move this PR along. Thank you!


.. important::
Before uninstalling a previous version of the Windows virtual :abbr:`IoT (Internet of Things)`
box, ensure that there is a newer version of Odoo's virtual :abbr:`IoT (Internet of Things)` box
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is referred to throughout as Windows virtual IoT box, so just changing to be consistent

Suggested change
box, ensure that there is a newer version of Odoo's virtual :abbr:`IoT (Internet of Things)` box
box, ensure that there is a newer version of Windows virtual :abbr:`IoT (Internet of Things)` box

Comment on lines 101 to 104
--> Windows` to view the date next to the :file:`odoo_(version).latest.exe` file. *(version)* is
equal to the version of Odoo (e.g. 16.0, 17.0). The latest version of the Windows virtual
:abbr:`IoT (Internet of Things)` box can be downloaded by selecting this file, or it is always
available at the `Odoo Download <https://odoo.com/download/>`_ page.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Matching the UI and fixing the sentence

Suggested change
--> Windows` to view the date next to the :file:`odoo_(version).latest.exe` file. *(version)* is
equal to the version of Odoo (e.g. 16.0, 17.0). The latest version of the Windows virtual
:abbr:`IoT (Internet of Things)` box can be downloaded by selecting this file, or it is always
available at the `Odoo Download <https://odoo.com/download/>`_ page.
--> windows/` to view the date next to the :file:`odoo_(version).latest.exe` file; where
*(version)* is equal to the version of Odoo (e.g. 16.0, 17.0). The latest version of the Windows
virtual :abbr:`IoT (Internet of Things)` box can be downloaded by selecting this file, or it is
always available at the `Odoo Download <https://odoo.com/download/>`_ page.

Comment on lines 126 to 127
(stable version) --> Windows` to and select the :file:`odoo_(version).latest.exe` file. *(version)*
is equal to the version of Odoo (e.g. 16.0, 17.0).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Matching UI and fixing sentence

Suggested change
(stable version) --> Windows` to and select the :file:`odoo_(version).latest.exe` file. *(version)*
is equal to the version of Odoo (e.g. 16.0, 17.0).
(stable version) --> windows/` to and select the :file:`odoo_(version).latest.exe` file; where
*(version)* is equal to the version of Odoo (e.g. 16.0, 17.0).

Comment on lines 129 to 131
To download from the :guilabel:`Odoo Download` page, find the :guilabel:`Odoo 16` section, or a
version higher (depending on what version the database is), and select the :guilabel:`Download`
button for :guilabel:`Windows`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can be simplified to

Suggested change
To download from the :guilabel:`Odoo Download` page, find the :guilabel:`Odoo 16` section, or a
version higher (depending on what version the database is), and select the :guilabel:`Download`
button for :guilabel:`Windows`.
To download from the :guilabel:`Odoo Download` page, find the section for the version of Odoo (e.g.
16.0, 17.0), and select the :guilabel:`Download` button for :guilabel:`Windows`.

@samueljlieber samueljlieber force-pushed the 16.0-general-iot-updating-windows-iot-tiku branch from e3f85e6 to 0e09ebf Compare March 28, 2024 18:26
Copy link
Contributor

@samueljlieber samueljlieber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Implemented my changes in 0e09ebf, approving. Thank you @tiku-odoo 👍
..
@robodoo r+

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants