Skip to content

Conversation

@nmarotte
Copy link
Contributor

No description provided.

@robodoo
Copy link
Collaborator

robodoo commented Sep 29, 2023

@nmarotte nmarotte force-pushed the 16.0-upgrade_doc_condensing-nama branch from 69523a3 to 806b219 Compare October 5, 2023 06:32
@nmarotte
Copy link
Contributor Author

nmarotte commented Oct 5, 2023

Hey @xpl-odoo @jcs-odoo , I've applied suggestions we discussed last time, except for a few TODO this is almost ready for review, would you mind checking it out to see how it is ?

Thanks a lot :)

@nmarotte nmarotte force-pushed the 16.0-upgrade_doc_condensing-nama branch from 16d7644 to a4500b2 Compare October 13, 2023 13:17
@nmarotte
Copy link
Contributor Author

Hello @xpl-odoo , I've applied your suggested changes and preemptively adapted the other upgrade page that was not reviewed yet (it is definitely not perfect but hopefully it should make your review faster haha). Don't hesitate to fix typos or simple rephrasing on the fly, if that's easier for you. Otherwise, don't worry I will take the time to fix them :)

Thanks again a lot for your help !

@avg-odoo
Copy link
Contributor

@xpl-odoo Hi - FYI level of the urgency has decreased here since we want our head of dept to review the content

Copy link
Contributor

@augustoPerez augustoPerez left a comment

Choose a reason for hiding this comment

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

Hi @nmarotte
I added the comments for the first part of the documentation
I excluded the Upgrade for developers page for the moment, as it needs to be refactored

Comment on lines 95 to 96
The :guilabel:`Upgrade` tab is available for valid projects with a valid production build
in the branches view.
Copy link
Contributor

Choose a reason for hiding this comment

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

It's not super clear what all this valid mean

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Part of the current documentation : https://www.odoo.com/documentation/16.0/administration/upgrade/odoo_sh.html#overview

I think it refers to the branch being up and running

Copy link
Contributor

Choose a reason for hiding this comment

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

IMO, it was meant for:
"valid projects": projects with a valid subscription (that are still paying), so for example no trial subscriptions, or educational ones (don't know if they exist on SH)
and "valid production build": what you said.

But it's still seems confusing.

I think we could ask STS (he developed the feature and knows the conditions he added in the code)

Hi @sts-odoo
Could you help us understand what you meant when you wrote this note in the documentation?
image
What does "valid project" and "valid production build" means?

Copy link
Contributor

Choose a reason for hiding this comment

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

@augustoPerez @nmarotte I confirm that it means a valid project (running odoo.sh subsrciption on odoo.com not a trial one, ie partnership), AND a valid subcription for the database (running based on odoo.com status).

In other places in the UI of Odoo.sh customer will see if their database is valid or not and why so it is clear enough. The documentation should be complementary to the UI not a copy or replacement of the messages that exist already in the UI.

@augustoPerez does the upgrade platform accepts non valid database to upgrade? If so, why, and do we want that as well on odoo.sh?

Copy link
Contributor Author

@nmarotte nmarotte Nov 2, 2023

Choose a reason for hiding this comment

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

Hey @sts-odoo , indeed the upgrade platform only accepts database with a valid subscription number, so no subscription, no request. I changed the note to

The :guilabel:`Upgrade` tab is available for projects with a valid subscription.

since, indeed, if the production branch is red/killed/dropped, it would make no sense to upgrade it. Thanks for your input :D

@nmarotte nmarotte force-pushed the 16.0-upgrade_doc_condensing-nama branch from eb06f9d to 27c76d1 Compare November 2, 2023 11:55
@nmarotte
Copy link
Contributor Author

nmarotte commented Nov 7, 2023

Merged in #6274

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants