Fix recursion in effect when fetching series acls

[dennis531]

Fixes the mentionied bug in #829.

Due to the React.stric mode, the ACLs are loaded twice and the notification is displayed twice, see https://react.dev/reference/react/StrictMode:

This problem could not be solved by simply putting dispatch(removeNotificationWizardForm()); at the beginning of the effect because the fetch is asynchronous. Is this a problem and should we fix it?

Close #829

[github-actions]

Use docker or podman to test this pull request locally.

Run test server using develop.opencast.org as backend:

podman run --rm -it -p 127.0.0.1:3000:3000 ghcr.io/opencast/opencast-admin-interface:pr-835

Specify a different backend like stable.opencast.org:

podman run --rm -it -p 127.0.0.1:3000:3000 -e PROXY_TARGET=https://stable.opencast.org ghcr.io/opencast/opencast-admin-interface:pr-835

It may take a few seconds for the interface to spin up.
It will then be available at http://127.0.0.1:3000.
For more options you can pass on to the proxy, take a look at the README.md.

[github-actions]

This pull request is deployed at test.admin-interface.opencast.org/835/2024-07-15_14-55-28/ .
It might take a few minutes for it to become available.

[Arnei]

Reasonable change and works.

While you are at it, could you remove the eslint disabling comment? Looks like it really should not be there.

[dennis531]

While you are at it, could you remove the eslint disabling comment? Looks like it really should not be there.

Done. I needed to add dispatch to the list of dependencies, otherwise the warning would be triggered.