change notification to react-toast-notification

[bestinbthomas]

No description provided.

[kumaran-14]

@shb9019

First off, there seems to be no enum for socket events and possible notifications. We should add them.
Secondly, Look into the Notification actions. Since the notification trigger logic is shifted to the useEffect method in the component, there seems to be few redundant actions. Should we keep them?

Also, review the pr and look into the dispatch sagas being called. Is the current way of calling them okay?

Minor stuff: The notifications in redux state is simply an array [], not an array of specific type. We should pick a data format and stick to it.

codecharacter-web/src/app/reducers/Notification.ts

Lines 31 to 33 in 0dd0421

	// @ts-ignore
	(notification) => notification.type !== action.payload.type,
	);

[kumaran-14]

@bestinbthomas We may need to work on this a bit more.