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

Refactor UI #14

Merged
merged 11 commits into from Dec 20, 2019
Merged

Refactor UI #14

merged 11 commits into from Dec 20, 2019

Conversation

overlineink
Copy link
Member

@overlineink overlineink commented Dec 20, 2019 •
edited

Bugs Fixed:

  • Fix table default empty message template;
  • Fix duplicated design from: contacts component;

Features:

Add search widget along side dynamic table;

PS: Change snapshotSerializers field on the main Jest configuration to get tests running properly according with this Issue on @nrwl/nx repo till this PR been merged.

snapshotSerializers: [
    'jest-preset-angular/build/AngularSnapshotSerializer.js',
    'jest-preset-angular/build/HTMLCommentSerializer.js'
  ]

@overlineink overlineink added PR status: tests needed priority: low Low priority to merging this PR labels Dec 20, 2019
@overlineink overlineink merged commit b5251bc into master Dec 20, 2019
@overlineink overlineink deleted the refactor-ui branch December 20, 2019 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
PR status: tests needed priority: low Low priority to merging this PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@overlineink