Skip to content

Conversation

@fw-bot
Copy link
Collaborator

@fw-bot fw-bot commented Sep 14, 2021

This is the V14 followup of #1107 . V14 has two different services which can be used, because Carvajal split into 2 entities.

Forward-Port-Of: #1125

The test server url for Carvajal T&S was out of date and has been
updated with the correct one. Carvajal has also split into 2
different companies, with new customers using the new company (CSC)
by default but the older company (CTS) still being available for
existing customers. See task 2508208. So the documentation was
updated to include both of them.

CTS (Carvajal T&S) was being used to refer to Carvajal in general,
so to avoid confusion those mentions have been changed to just
"Carvajal" since it can refer to either CTS or CSC.

X-original-commit: 396df2a
@robodoo
Copy link
Collaborator

robodoo commented Sep 14, 2021

@fw-bot
Copy link
Collaborator Author

fw-bot commented Sep 14, 2021

This PR targets 15.0 and is part of the forward-port chain. Further PRs will be created up to master.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@AntoineVDV
Copy link
Collaborator

@fw-bot r+

@raf-odoo
Copy link
Contributor

@AntoineVDV Any idea why this one got stuck?

@raf-odoo
Copy link
Contributor

Seems the CI step isn't checked in the pull request dashboard for some reason.

@AntoineVDV
Copy link
Collaborator

@AntoineVDV Any idea why this one got stuck?

There are issues with the bots atm. No big deal since we want the change to be merged into saas-14.5 which will overwrite 15.0 after the OXP.

@AntoineVDV AntoineVDV closed this Sep 20, 2021
@raf-odoo
Copy link
Contributor

@AntoineVDV I suppose that means I'll need to create a separate PR for master, since the chain breaks at this point?

@AntoineVDV
Copy link
Collaborator

@AntoineVDV I suppose that means I'll need to create a separate PR for master, since the chain breaks at this point?

No need, I'll manually forward-port 15.0 to master after the OXP.

@raf-odoo
Copy link
Contributor

Ok, thanks :-)

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.

5 participants