crm icon indicating copy to clipboard operation
crm copied to clipboard

[18.0][MIG] crm_won_reason: Migration to 18.0

Open apami0111 opened this issue 6 months ago • 3 comments

  • Migrated from 15.0 to 18.0.
  • Replaced deprecated attrs with new invisible expression in crm_won_reason/views/crm_views.xml:
  • Updated test class inheritance from "TransactionCase" to "BaseCommon".

apami0111 avatar Jun 28 '25 15:06 apami0111

@BhaveshHeliconia Thank you for the feedback. I have reviewed the case and confirm that the crm_won_reason module by itself does not present the issue of two "Won" buttons. This behavior only occurs when it is installed together with the crm_won_restrict_per_stage module, as both apply an invisible attribute to the same button, causing the view to be triggered and resulting in two instances of the button being displayed. In an environment with only crm_won_reason, both functionality and interface work as expected. Captura desde 2025-08-07 21-44-26

apami0111 avatar Aug 12 '25 01:08 apami0111

@apami0111 : The functionality works perfectly, but having the same "Won" label for both buttons can be confusing.

BhaveshHeliconia avatar Aug 12 '25 03:08 BhaveshHeliconia

/ocabot migration crm_won_reason

pedrobaeza avatar Oct 24 '25 02:10 pedrobaeza

ping @pedrobaeza

celm1990 avatar Nov 21 '25 02:11 celm1990

@BhaveshHeliconia has requested changes

pedrobaeza avatar Nov 21 '25 07:11 pedrobaeza

@pedrobaeza Thanks for the note. The issue mentioned in

@BhaveshHeliconia has requested changes is already resolved. As I explained here: https://github.com/OCA/crm/pull/658#issuecomment-3177391757

the duplicated “Won” button was not caused by crm_won_reason, but by its interaction with crm_won_restrict_per_stage, since both apply an invisible attribute to the same button.

With only crm_won_reason installed, everything works correctly.

apami0111 avatar Nov 28 '25 02:11 apami0111

Yeah, but he should confirm it.

pedrobaeza avatar Nov 28 '25 06:11 pedrobaeza