Skip to content

[19.0][OU-IMP] crm: Add separator to phone and mobile concatenation#5763

Open
christian-ramos-tecnativa wants to merge 1 commit into
OCA:19.0from
Tecnativa:19.0-ou-crm-mobile
Open

[19.0][OU-IMP] crm: Add separator to phone and mobile concatenation#5763
christian-ramos-tecnativa wants to merge 1 commit into
OCA:19.0from
Tecnativa:19.0-ou-crm-mobile

Conversation

@christian-ramos-tecnativa

Copy link
Copy Markdown

Depending on how the user wrote down the phone and mobile fields, after the migration the phone can get ambiguous. Added a separator to avoid it.

@Tecnativa
@pedrobaeza @eduezerouali-tecnativa

@OCA-git-bot OCA-git-bot added mod:openupgrade_scripts Module openupgrade_scripts series:19.0 labels Jun 29, 2026
@pedrobaeza pedrobaeza added this to the 19.0 milestone Jun 29, 2026

@pedrobaeza pedrobaeza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

As you are here, please make a copy of the column first for preserving original information.

@christian-ramos-tecnativa

Copy link
Copy Markdown
Author

Do you mean creating an extra column like original_phone with the info in phone column?

@pedrobaeza

Copy link
Copy Markdown
Member

Use copy_columns method, and pass None to get an automatic backup column name.

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

Labels

mod:openupgrade_scripts Module openupgrade_scripts series:19.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants