fix: so previous change id is used on delete change for notification template

[davidsoderberg]

What change does this PR introduce?

Same behaviour as on update notification template

Why was this change needed?

To fix so template is deleted in prod env when using promote all

[linear]

NV-1621 🐛 Bug Report: Admin UI not deleting templates in production

📜 Description

When the user promotes development changes to production. All changes are not promoted correctly. There is some mismatch.

👟 Reproduction steps

Create a few notification templates in the development environment
Then promote those changes to production.
After successful promotion, delete one or two templates from development.
Now promote changes again to production.
Deleted templates are still there in prod and non-editable.

👍 Expected behavior

It should delete it

👎 Actual Behavior with Screenshots

💻 Operating system

Windows

🤖 Node Version

12

📃 Provide any additional context for the Bug.

dima fix it by deleting it manually from backend. (if anyone has the same issue)

👀 Have you spent some time to check if this bug has been raised before?

• I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

• I have read the Contributing Guidelines

Are you willing to submit PR?

Yes I am willing to submit a PR!

[ainouzgali]

Is it possible to add a test? Or will it be too flaky?

[davidsoderberg]

Is it possible to add a test? Or will it be too flaky?

I am looking into this right now 😄