-
Notifications
You must be signed in to change notification settings - Fork 191
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
fixed total no of posts not updating on deleting a post fixes #3326 #1443
Conversation
@rohit645 Thank you for your contribution! I will review shortly! |
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.
Thank you for fixing this! Its awesome! Just a super-small thing, I don't think the linter will let the commit through otherwise, could you remove the extra spaces i noted in the code?
@@ -40,7 +40,8 @@ function PostActionsDirective( | |||
function activate() { | |||
$scope.statuses = PostActionsService.getStatuses(); | |||
} | |||
|
|||
|
|||
// Called when a single data item is deleted. |
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.
Thank you for figuring this out and clarifying it in the comments!
7020c89
to
552f742
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! Thank you @rohit645
@Angamanga To merge to develop |
This pull request makes the following changes:
Testing checklist:
Fixes ushahidi/platform#3326 .