Skip to content

[18.0][MIG] srm: Migration to 18.0#632

Open
vvrossem wants to merge 3 commits into
OCA:18.0from
camptocamp:18.0-add-srm
Open

[18.0][MIG] srm: Migration to 18.0#632
vvrossem wants to merge 3 commits into
OCA:18.0from
camptocamp:18.0-add-srm

Conversation

@vvrossem
Copy link
Copy Markdown

Initial (closed) 14.0 PR: #448

Comment thread srm/views/srm_lead.xml
@@ -65,34 +64,44 @@
expr="//button[@name='action_sale_quotations_new']"
position="attributes"
>
<!-- addition to `type == 'lead' or probability == 0 and not active` -->
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

thx for comments

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Oct 5, 2025

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions Bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Oct 5, 2025
@github-actions github-actions Bot closed this Nov 9, 2025
@vvrossem
Copy link
Copy Markdown
Author

vvrossem commented Mar 5, 2026

hello @OCA/crm-sales-marketing-maintainers, could you please reopen this PR and set the no stale label? 🙏

@simahawk simahawk reopened this Mar 5, 2026
@simahawk simahawk removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Mar 5, 2026
Comment thread srm/models/crm_lead.py Outdated
@@ -0,0 +1,81 @@
# Copyright 2022 Telmo Santos <telmo.santos@camptocamp.com>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
# Copyright 2022 Telmo Santos <telmo.santos@camptocamp.com>
# Copyright 2022 Camptocamp SA
# @author Telmo Santos <telmo.santos@camptocamp.com>

everywhere

Comment thread srm/models/crm_lead.py Outdated

def action_rfq_new(self):
action = self.env["ir.actions.actions"]._for_xml_id("srm.action_lead_rfq_new")
action["context"] = {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

you should preserve the current context IMO

Comment thread srm/models/purchase.py Outdated
"crm.lead",
string="Opportunity",
check_company=True,
domain="[('type', '=', 'opportunity'), ('request_type', '=', 'supplier'), '|', ('company_id', '=', False), ('company_id', '=', company_id)]", # noqa
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

why noqa?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Comment thread srm/views/srm_lead.xml
<!-- Lead Menu -->
<record id="srm_lead_action_pipeline" model="ir.actions.act_window">
<field name="name">Srm: Pipeline</field>
<field name="res_model">crm.lead</field>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

this file is a mix of views for different models... does it make sense?

Comment thread srm/wizard/srm_opportunity_to_rfq.py Outdated
Comment thread srm/wizard/srm_opportunity_to_rfq.py Outdated
Comment thread srm/README.rst Outdated
=========================================
Sequential Code for Leads / Opportunities
=========================================
============
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

use README fragments

Comment thread srm/__manifest__.py Outdated
"author": "Camptocamp, Odoo Community Association (OCA)",
"summary": "Use CRM model for suppliers",
"license": "AGPL-3",
"category": "CRM",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

set dev status to Alpha (there's no test, for starting)

@vvrossem vvrossem force-pushed the 18.0-add-srm branch 6 times, most recently from 8ba2eb0 to b7629ae Compare March 6, 2026 15:41
@yankinmax
Copy link
Copy Markdown

Hello @simahawk can you pls take a look again, it seems to be fixed?

@flotho
Copy link
Copy Markdown
Member

flotho commented May 20, 2026

Hi @vvrossem , any chance to rebase ?
Thanks

@yankinmax
Copy link
Copy Markdown

Hi @vvrossem , any chance to rebase ? Thanks

Rebased, thanks

@vvrossem
Copy link
Copy Markdown
Author

Hi @vvrossem , any chance to rebase ? Thanks

Rebased, thanks

Thank you 🙏 !

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants