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.

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

add chip and date format in root table #1638

Merged
merged 2 commits into from
Jun 12, 2024

Conversation

danoswaltCL
Copy link
Collaborator

went ahead and added in chip and fixed date formatting in root table rows

image

@danoswaltCL danoswaltCL requested review from bcb37, Yagnik56 and KD1712 June 10, 2024 19:41
import { MatTooltipModule } from '@angular/material/tooltip';
import { TranslateModule } from '@ngx-translate/core';
import { MatChipsModule } from '@angular/material/chips';
import { RouterModule } from '@angular/router';
import { CommonStatusIndicatorChipComponent } from '../../../../../../../../shared-standalone-component-lib/components';
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there an easy way to get rid of these confusing and brittle relative import paths? I tried a few things with path aliases in ts-config, but couldn't find anything simple that worked.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes there should be, maybe we can spawn a quick task for that.

Copy link
Collaborator

@Yagnik56 Yagnik56 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good to me.

@danoswaltCL danoswaltCL merged commit ff3c924 into dev Jun 12, 2024
8 checks passed
@danoswaltCL danoswaltCL deleted the feature/root-table-date-format-and-chip branch June 12, 2024 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants