Describe the pull request
This pull request is dedicated to a comprehensive overhaul of the current notification system to leverage React's Portal system and to better handle Apollo errors. By integrating React Portal, we can benefit from its capability to render children into a DOM node that exists outside the root app and prevent rerender on show and hide notification. Additionally, the improved handling of Apollo errors will ensure more robust error detection and management within the notification system. Through this significant rework, we aim to improve the reliability, performance, and user experience of the notification system.
Checklist
[x] I have made the modifications or added tests related to my PR
[x] I have run the tests and linters locally and they pass
[x] I have added/updated the documentation for my RP
Show Validation
```
Success! The configuration is valid.
```
Terraform Plan 📖 success
Show Plan
```
kubernetes_config_map.stud42_config: Refreshing state... [id=sandbox/stud42-config]
module.istio.kubectl_manifest.virtual_services["dev-s42-sandbox"]: Refreshing state... [id=/apis/networking.istio.io/v1alpha3/namespaces/sandbox/virtualservices/dev-s42-sandbox]
module.jwtks_service.kubernetes_service.app[0]: Refreshing state... [id=sandbox/jwtks-service]
module.jwtks_service.kubernetes_deployment.app[0]: Refreshing state... [id=sandbox/jwtks-service]
module.jwtks_service.kubernetes_horizontal_pod_autoscaler_v2.app[0]: Refreshing state... [id=sandbox/jwtks-service]
module.jwtks_service.kubernetes_manifest.certificate["grpc-internal"]: Refreshing state...
No changes. Your infrastructure matches the configuration.
Terraform has compared your real infrastructure against your configuration
and found no differences, so no changes are needed.
```
Describe the pull request This pull request is dedicated to a comprehensive overhaul of the current notification system to leverage React's Portal system and to better handle Apollo errors. By integrating React Portal, we can benefit from its capability to render children into a DOM node that exists outside the root app and prevent rerender on show and hide notification. Additionally, the improved handling of Apollo errors will ensure more robust error detection and management within the notification system. Through this significant rework, we aim to improve the reliability, performance, and user experience of the notification system.
Checklist