-
Notifications
You must be signed in to change notification settings - Fork 282
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
ability to add members by searching for their names #2272
Conversation
Signed-off-by: Dvir Guttman <dvir.guttman@verizonmedia.com>
bebef12
to
ecf2d2f
Compare
Signed-off-by: Dvir Guttman <dvir.guttman@verizonmedia.com>
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.
Instead of making API calls for each keystroke, would it be better to store the userdata in redux layer and then return from there?
I think it will increase performance.
Signed-off-by: Dvir Guttman <dvir.guttman@verizonmedia.com>
Signed-off-by: Dvir Guttman <dvir.guttman@verizonmedia.com>
@@ -77,5 +81,26 @@ function refreshUserData(config, extServiceClient) { | |||
}); | |||
} | |||
|
|||
async function readUsersFileFromDisk(fileName) { |
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.
couldn't find use of this function, should it be removed?
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.
it allows alternative way to load user file from disk, to be used by local development (for example vzm-ui)
Signed-off-by: Dvir Guttman <dvir.guttman@verizonmedia.com>
Signed-off-by: Dvir Guttman <dvir.guttman@verizonmedia.com>
No description provided.