-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Refactor Workspace Invite Member page #5726
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
Changes from all commits
b7fb045
1d46238
4a8161c
5eb901e
d2bf778
668722e
e0d5e6b
786e1d9
034df24
273d81f
739cc4e
f66dd56
3bb2c88
eda25c8
74fa0d5
ef2e72c
76ca556
add3414
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -248,7 +248,7 @@ function invite(logins, welcomeNote, policyID) { | |
policy.alertMessage = ''; | ||
|
||
// Optimistically add the user to the policy | ||
Onyx.set(key, policy); | ||
Onyx.merge(key, policy); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I had to make this change otherwise, newly added members were being removed as soon as they were added to Onyx. I am not sure how to debug the Onyx but I tried debugging the code and somehow internal Keychanged handler is getting old value in the next tick. |
||
|
||
// Make the API call to merge the login into the policy | ||
API.Policy_Employees_Merge({ | ||
|
This file was deleted.
Uh oh!
There was an error while loading. Please reload this page.