-
Notifications
You must be signed in to change notification settings - Fork 47
Open
Labels
complexity:mediumSomething that requires one or few days to fixSomething that requires one or few days to fixkind:debtTechnical debtTechnical debtpriority:mediumMedium priority issues, should only be postponed if no other optionMedium priority issues, should only be postponed if no other optionseverity:minorNo impact, or minor impact, on live deploymentsNo impact, or minor impact, on live deploymentstopic:uiUI-related issuesUI-related issues
Description
Component:
UI, typing
Why this is needed:
This brings typesafety and enhanced developer experience to the project.
What should be done:
Add @flow annotations in ui/src/ducks/app/monitoring.js and fix all the linting issues raised. Try not using any keyword.
Test plan:
Run flow cli to ensure typing is correctly interpreted, navigate through the code base in your IDE and see if all references to monitoring reducer are correctly typed
Metadata
Metadata
Assignees
Labels
complexity:mediumSomething that requires one or few days to fixSomething that requires one or few days to fixkind:debtTechnical debtTechnical debtpriority:mediumMedium priority issues, should only be postponed if no other optionMedium priority issues, should only be postponed if no other optionseverity:minorNo impact, or minor impact, on live deploymentsNo impact, or minor impact, on live deploymentstopic:uiUI-related issuesUI-related issues