I10962 stable 3 5 0 fix

[Hafsa-Naeem]

No description provided.

[touhidurabir]

If I understand it correctly , here upgrading from min 3.1.0.0 to max 3.4.0.1 and upgrading from

1. 3.3.0-x to 3.5.0-x
2. 3.4.0-x to 3.5.0-x
3. 3.5.0-0/3.5.0-1 to 3.5.0-x

will not have this block of migration to run . so do we need it here ? please correct me if I am mistaken .

[touhidurabir]

here we any installation upgrading from

1. 3.3.0-0 to 3.5.0-x
2. 3.4.0-x to 3.5.0-x

will have this migration applied . So will it work if we move it under the block of <upgrade minversion="3.1.0.0" maxversion="3.4.9.9"> ?

[touhidurabir]

@Hafsa-Naeem Seems all ok . But I have some concerns regarding to upgrade path and blocks where we applied the migration, please check the comments .

[Hafsa-Naeem]

@touhidurabir Thanks, yes we don’t need a separate 3.3.0.0 - 3.4.9.9 block. To make the intent clear and avoid duplication, I've kept I10962_UpdateEmailTemplateVariables in two places only now:

3.1.0.0 - 3.4.9.9 (covers upgrades from ≤3.4)

3.5.0.0 - 3.5.9.9 (covers 3.5.0-0/3.5.0-1 -> 3.5.x)

This matches what I tested for 3.3.x -> 3.5.x, 3.4.x -> 3.5.x, and 3.5.0-0 -> 3.5.x.

[touhidurabir]

@Hafsa-Naeem , I have also approved the OMP and OPS PRs .