Skip to content

[19.0][ADD] crm_lead_phone_secondary#2270

Open
AndreuOForgeFlow wants to merge 2 commits into
OCA:19.0from
ForgeFlow:19.0-add-crm_lead_phone_secondary
Open

[19.0][ADD] crm_lead_phone_secondary#2270
AndreuOForgeFlow wants to merge 2 commits into
OCA:19.0from
ForgeFlow:19.0-add-crm_lead_phone_secondary

Conversation

@AndreuOForgeFlow
Copy link
Copy Markdown

@AndreuOForgeFlow AndreuOForgeFlow force-pushed the 19.0-add-crm_lead_phone_secondary branch 2 times, most recently from 45c9532 to 0588a05 Compare January 21, 2026 10:52
@AndreuOForgeFlow AndreuOForgeFlow marked this pull request as ready for review January 21, 2026 10:56
Copy link
Copy Markdown
Contributor

@BhaveshHeliconia BhaveshHeliconia left a comment

Choose a reason for hiding this comment

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

Remove redundant hasattr and getattr checks

@BhaveshHeliconia
Copy link
Copy Markdown
Contributor

phone2 field is not being set in contact when contact is created from lead and it is being cleared when contact is deleted

@AndreuOForgeFlow AndreuOForgeFlow force-pushed the 19.0-add-crm_lead_phone_secondary branch from 0588a05 to bb200bf Compare June 3, 2026 14:57
@OCA-git-bot OCA-git-bot added series:19.0 mod:crm_lead_phone_secondary Module crm_lead_phone_secondary labels Jun 3, 2026
Copy link
Copy Markdown
Contributor

@BhaveshHeliconia BhaveshHeliconia left a comment

Choose a reason for hiding this comment

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

LGTM!

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

Labels

mod:crm_lead_phone_secondary Module crm_lead_phone_secondary series:19.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants