-
Notifications
You must be signed in to change notification settings - Fork 77
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
feat(communities): Process declined requests #8056
Conversation
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.
Looks good. Just nit comments.
Make sure to assign a QA to this since it's a new feature
self.myCommunityRequests.add(membershipRequest) | ||
self.events.emit(SIGNAL_COMMUNITY_MY_REQUEST_REJECTED, CommunityRequestArgs(communityRequest: membershipRequest)) | ||
singletonInstance.globalEvents.myRequestToJoinCommunityRejected("Community Request Rejected", | ||
fmt "Your request to join community is rejected", membershipRequest.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.
nit: can you add an indent on those two lines to show they are the same as the event
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.
fixed
self.myCommunityRequests.add(communityRequest) | ||
self.events.emit(SIGNAL_COMMUNITY_MY_REQUEST_ADDED, CommunityRequestArgs(communityRequest: communityRequest)) | ||
except Exception as e: | ||
error "Error fetching my community requests", msg = e.msg |
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.
error "Error fetching my community requests", msg = e.msg | |
error "Error fetching my declined community requests", msg = e.msg |
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.
fixed
Jenkins BuildsClick to see older builds (15)
|
990db05
to
27cbeb1
Compare
27cbeb1
to
87c5380
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.
Looks good
87c5380
to
3764681
Compare
3764681
to
f7768c7
Compare
@jrainville @iurimatias do we want this feature to be landed or not? is it still in scope of 0.9.0? |
@jrainville please revisit this, i am not sure we still want it, also the PR is really old. I removed the 0.9.0 milestone for the ticket |
Closing as we do not want this in the end |
Part of: #7072
fixes #8316
StatusGo PR here
What does the PR do
Process declined community requests
Affected areas
Membership requests
Screenshot of functionality (including design for comparison)
decline_request.mov