-
Notifications
You must be signed in to change notification settings - Fork 6
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
ft add client registry #32
ft add client registry #32
Conversation
jabahum
commented
Jul 12, 2024
- This PR adds client registry push and pulling of data
f939748
to
5fe1cad
Compare
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.
Code looks good besides a few cleanups, I am not sure how to test this visually
src/client-registry/client-registry-data/client-registry-data.component.tsx
Show resolved
Hide resolved
src/client-registry/client-registry-data/client-registry-data.scss
Outdated
Show resolved
Hide resolved
...egistry/client-registry-data/client-registry-dialogs/send-patient-to-cr-dialog.component.tsx
Outdated
Show resolved
Hide resolved
...egistry/client-registry-data/client-registry-dialogs/send-patient-to-cr-dialog.component.tsx
Outdated
Show resolved
Hide resolved
e88a21a
to
9cd1dbc
Compare
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, just a few things to clear
src/client-registry/client-registry-data/client-registry-data.component.tsx
Show resolved
Hide resolved
|
||
.arrowIcon { | ||
fill: var(--cds-link-primary, #0f62fe) !important; | ||
} |
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.
@jabahum the new line issues seem to still be existing