Skip to content

Conversation

@mao-odoo
Copy link
Contributor

@mao-odoo mao-odoo commented Aug 11, 2022

Correction to a typo introduced in #2133 that made the line completly nonsensical and made
people think that they could migrate just part of the database.

opw-2947684

Correction to a typo that made the line completly nonsensical and made
people think that they could migrate just part of the database.

opw-2947684
@mao-odoo mao-odoo requested a review from jaae-odoo August 11, 2022 14:07
@robodoo
Copy link
Collaborator

robodoo commented Aug 11, 2022

@C3POdoo C3POdoo requested a review from a team August 11, 2022 14:09
Copy link
Contributor

@jaae-odoo jaae-odoo left a comment

Choose a reason for hiding this comment

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

@mao-odoo: Indeed, well spotted!

@mao-odoo
Copy link
Contributor Author

@jaae-odoo Do you know who is in charge of doing the
"""
@robodoo r+
"""
for the documentation?

@robodoo
Copy link
Collaborator

robodoo commented Aug 12, 2022

I'm sorry, @mao-odoo. You can't review+.

@jcs-odoo jcs-odoo changed the title maintain : odoosh to online : branch vs brand [IMP] maintain : process for odoo.sh to odoo online Aug 18, 2022
@jcs-odoo
Copy link
Contributor

@robodoo r+

robodoo pushed a commit that referenced this pull request Aug 18, 2022
Correction to a typo that made the line completly nonsensical and made
people think that they could migrate just part of the database.

opw-2947684

closes #2583

Signed-off-by: Castillo Jonathan (jcs) <jcs@odoo.com>
@jcs-odoo
Copy link
Contributor

@jaae-odoo Do you know who is in charge of doing the """ @robodoo r+ """ for the documentation?

@mao-odoo , Jason forgot to add the "next reviewer" who would have done the r+ . Normally you don't have to worry about the reviewers and the r+ once you put a PR ready for review.

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.

4 participants