POLIO-1770, POLIO-1768: delete unused scopes + migration

[quang-le]

What problem is this PR solving? Explain here in one sentence.

Related JIRA tickets : POLIO-1770, POLIO-1768

Self proofreading checklist

• Did I use eslint and black formatters
• Is my code clear enough and well documented
• Are my typescript files well typed
• New translations have been added or updated if new strings have been introduced in the frontend
• My migrations file are included
• Are there enough tests
• Documentation has been included (for new feature)

Doc

N/A

Changes

• POLIO-1768: migration

  • Add migration that deletes unused scopes
  • Note: rounds w/o campaign were not deleted

• POLIO-1770: update API

  • Check existing scopes:
    • if campaign switches to round scope, delete campaign scope and vice-versa
    • if scope per campaign, ignore scopes per round received from front end and vice versa
    • Add tests

How to test

On polio DB:

• Go to campaigns
• Open a campaign
• Go to Scopes tab
• Tick "separate scopes per round" box
• Add new scope
• Save

--> The old scopes should have been removed from GET response, but will still be visible in front-end because of formik state. Reloading the page will make it diappear

• Repeat the operation the other way around

Print screen / video

Uploading Screen Recording 2024-11-29 at 15.13.32.mov…

Notes

MERGES INTO #1830

Follow the Conventional Commits specification

The merge message of a pull request must follow the Conventional Commits specification.

This convention helps to automatically generate release notes.

Use lowercase for consistency.

Example:

fix: empty instance pop up

Refs: IA-3665

Note that the Jira reference is preceded by a line break.

Both the line break and the Jira reference are entered in the Add an optional extended description… field.

[kemar]

LGTM

The logic seems good and kudos for the test which helps to understand the code.

I suggest some simplifications to make the code easier to read. You're free to apply them or not.