-
Notifications
You must be signed in to change notification settings - Fork 78
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
fix ImportCommunityPopup
issues, remove private key importing
#12554
Conversation
Jenkins BuildsClick to see older builds (7)
|
76f6596
to
c47dc02
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.
Overall LGTM, just some minor points
return Utils.getCommunityDataFromSharedLink(inputKey).communityId; | ||
if (Utils.isStatusDeepLink(inputKey)) { | ||
const linkData = Utils.getCommunityDataFromSharedLink(inputKey) | ||
return !linkData ? "" : linkData.communityId |
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.
return !linkData ? "" : linkData.communityId | |
return !!linkData ? linkData.communityId : "" |
I always find this double negative idiom ugly :)
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.
I don't get it, you find it ugly, but still suggest? 😄
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
Fixes #12465
Fixes #12542
Drops the usage of
parseCommunitySharedUrl
(#12356)What does the PR do
requestCommunityInfo
callreuestCommunityInfo
fromgetCommunityDetails
.Because the former returns nothing and requires binding to nim signals.
ImportCommunityPopup
Affected areas
ImportCommunityPopup
Screenshot of functionality (including design for comparison)
Screen.Recording.2023-10-26.at.11.02.21.mov
Screen.Recording.2023-10-26.at.11.05.15.mov