Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove unecessary error management in ui #3320

Closed
freddidierRTE opened this issue Jul 13, 2022 · 0 comments · Fixed by #3473
Closed

Remove unecessary error management in ui #3320

freddidierRTE opened this issue Jul 13, 2022 · 0 comments · Fixed by #3473
Assignees
@quinarygio
Labels
Technical debt

Comments

@freddidierRTE
Copy link
Contributor

freddidierRTE commented Jul 13, 2022 •
edited
Loading

Remove all classes ui/main/src/app/common/error that are not useful

You will need to replace all `handleError' call in subclasses by a meaningfull log using LoggerService and/or a message in orange on top of the screen using existing alert component
@quinarygio quinarygio self-assigned this Aug 22, 2022
@quinarygio quinarygio moved this from 🔖 To do to 🏗 In progress in OperatorFabric Release 3.10.0 Aug 22, 2022
quinarygio added a commit that referenced this issue Aug 23, 2022
@quinarygio
Remove unecessary error management in ui (#3320)
3c0be6c 
Signed-off-by: Giovanni Ferrari <[email protected]>
@quinarygio quinarygio mentioned this issue Aug 23, 2022
Merged
@quinarygio quinarygio moved this from 🏗 In progress to 👀 To review in OperatorFabric Release 3.10.0 Aug 23, 2022
vlo-rte pushed a commit that referenced this issue Aug 26, 2022
@quinarygio @vlo-rte
Remove unecessary error management in ui (#3320)
d674112 
Signed-off-by: Giovanni Ferrari <[email protected]>
Repository owner moved this from 👀 To review to ✅ Done in OperatorFabric Release 3.10.0 Aug 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@quinarygio quinarygio

Labels
Technical debt
Projects
No open projects
OperatorFabric Release 3.10.0
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove unecessary error management in ui (#3320) opfab/operatorfabric-core
2 participants
@quinarygio @freddidierRTE