Skip to content

Conversation

@chiaraprattico
Copy link
Contributor

Deleting the Odoo Tutorials Invoices link since the course has been merged, and the link isn't necessary anymore.

@chiaraprattico chiaraprattico requested a review from jcs-odoo June 20, 2022 13:15
@robodoo
Copy link
Collaborator

robodoo commented Jun 20, 2022

Copy link
Contributor

@jcs-odoo jcs-odoo left a comment

Choose a reason for hiding this comment

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

Hello @chiaraprattico

  • ici, le tag du commit message c'est plutôt [IMP]
  • La petite explication que tu as mise dans la pr, c'est mieux de le mettre dans la suite du message commit

Mais c'est franchement pas très important pour un tout petit commit comme ça :)

@jcs-odoo jcs-odoo requested a review from a team June 20, 2022 14:34
Copy link
Collaborator

@Feyensv Feyensv left a comment

Choose a reason for hiding this comment

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

robodoo r+

@Feyensv Feyensv changed the title [REM] accounting: removal odoo tutorials invoices link [REM] accounting: remove the odoo tutorials invoices link Jun 20, 2022
@Feyensv
Copy link
Collaborator

Feyensv commented Jun 20, 2022

@robodoo squash

robodoo pushed a commit that referenced this pull request Jun 20, 2022
Deleting the Odoo Tutorials Invoices link since the course has been merged, and the link isn't necessary anymore.

closes #2243

Signed-off-by: Victor Feyens (vfe) <[email protected]>
@Feyensv Feyensv changed the title [REM] accounting: remove the odoo tutorials invoices link [IMP] accounting: remove the odoo tutorials invoices link Jun 20, 2022
@robodoo
Copy link
Collaborator

robodoo commented Jun 20, 2022

Merge method set to squash

@Feyensv
Copy link
Collaborator

Feyensv commented Jun 20, 2022

Well, latest change to the title was too late, not a big problem anyway :D

@robodoo robodoo closed this Jun 20, 2022
@robodoo robodoo temporarily deployed to merge June 20, 2022 15:38 Inactive
robodoo pushed a commit that referenced this pull request Jun 20, 2022
Deleting the Odoo Tutorials Invoices link since the course has been merged, and the link isn't necessary anymore.

closes #2248

Forward-port-of: #2243
Signed-off-by: Victor Feyens (vfe) <[email protected]>
robodoo pushed a commit that referenced this pull request Jun 20, 2022
Deleting the Odoo Tutorials Invoices link since the course has been merged, and the link isn't necessary anymore.

closes #2247

Forward-port-of: #2243
Signed-off-by: Victor Feyens (vfe) <[email protected]>
robodoo pushed a commit that referenced this pull request Jun 20, 2022
Deleting the Odoo Tutorials Invoices link since the course has been merged, and the link isn't necessary anymore.

closes #2246

Forward-port-of: #2243
Signed-off-by: Victor Feyens (vfe) <[email protected]>
robodoo pushed a commit that referenced this pull request Jun 20, 2022
Deleting the Odoo Tutorials Invoices link since the course has been merged, and the link isn't necessary anymore.

closes #2244

Forward-port-of: #2243
Signed-off-by: Victor Feyens (vfe) <[email protected]>
robodoo pushed a commit that referenced this pull request Jun 20, 2022
Deleting the Odoo Tutorials Invoices link since the course has been merged, and the link isn't necessary anymore.

closes #2245

Forward-port-of: #2243
Signed-off-by: Victor Feyens (vfe) <[email protected]>
robodoo pushed a commit that referenced this pull request Jun 20, 2022
Deleting the Odoo Tutorials Invoices link since the course has been merged, and the link isn't necessary anymore.

closes #2250

Forward-port-of: #2243
Signed-off-by: Victor Feyens (vfe) <[email protected]>
robodoo pushed a commit that referenced this pull request Jun 20, 2022
Deleting the Odoo Tutorials Invoices link since the course has been merged, and the link isn't necessary anymore.

closes #2249

Forward-port-of: #2243
Signed-off-by: Victor Feyens (vfe) <[email protected]>
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.

5 participants