-
Notifications
You must be signed in to change notification settings - Fork 353
Conversation
ocket8888
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Most - but not all - of your linting errors can be fixed by running ng lint --fix
experimental/traffic-portal/src/app/core/statuses/status-details/status-details.component.html
Outdated
Show resolved
Hide resolved
experimental/traffic-portal/src/app/core/statuses/status-details/status-details.component.html
Show resolved
Hide resolved
experimental/traffic-portal/src/app/core/statuses/statuses-table/statuses-table.component.ts
Outdated
Show resolved
Hide resolved
experimental/traffic-portal/src/app/core/statuses/statuses-table/statuses-table.component.ts
Outdated
Show resolved
Hide resolved
experimental/traffic-portal/src/app/core/statuses/statuses-table/statuses-table.module.ts
Outdated
Show resolved
Hide resolved
experimental/traffic-portal/src/app/shared/navigation/navigation.service.ts
Outdated
Show resolved
Hide resolved
ocket8888
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still has a lot of linting violations. If you've fixed everything that can be automatically fixed with ng lint --fix, ng lint (without --fix) will list the remaining problems for you
experimental/traffic-portal/src/app/core/statuses/status-details/status-details.component.html
Outdated
Show resolved
Hide resolved
experimental/traffic-portal/src/app/core/statuses/status-details/status-details.component.ts
Outdated
Show resolved
Hide resolved
...imental/traffic-portal/src/app/core/statuses/statuses-table/statuses-table.component.spec.ts
Outdated
Show resolved
Hide resolved
...imental/traffic-portal/src/app/core/statuses/status-details/status-details.component.spec.ts
Outdated
Show resolved
Hide resolved
...imental/traffic-portal/src/app/core/statuses/status-details/status-details.component.spec.ts
Outdated
Show resolved
Hide resolved
experimental/traffic-portal/src/app/core/statuses/statuses-table/statuses-table.component.ts
Outdated
Show resolved
Hide resolved
experimental/traffic-portal/src/app/core/statuses/status-details/status-details.component.ts
Outdated
Show resolved
Hide resolved
experimental/traffic-portal/src/app/core/statuses/status-details/status-details.component.html
Outdated
Show resolved
Hide resolved
ocket8888
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs to be rebased on the latest changes in master
experimental/traffic-portal/src/app/core/statuses/status-details/status-details.component.ts
Outdated
Show resolved
Hide resolved
experimental/traffic-portal/src/app/core/statuses/status-details/status-details.component.ts
Outdated
Show resolved
Hide resolved
experimental/traffic-portal/src/app/core/statuses/status-details/status-details.component.ts
Outdated
Show resolved
Hide resolved
...imental/traffic-portal/src/app/core/statuses/statuses-table/statuses-table.component.spec.ts
Outdated
Show resolved
Hide resolved
...imental/traffic-portal/src/app/core/statuses/status-details/status-details.component.spec.ts
Outdated
Show resolved
Hide resolved
rimashah25
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
This PR is not related to any issue.
Which Traffic Control components are affected by this PR?
What is the best way to verify this PR?
Tried to develop a parity of TPV1 - statuses page.
PR submission checklist