-
Notifications
You must be signed in to change notification settings - Fork 506
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
When you delete a post in dataview, the total number of posts is not updated #3326
Comments
Hey! I am Sargam, an outreachy applicant. I would like to go ahead with this issue. Thanks:) |
@FabSar Please go ahead! Just comment here if you need guidance! |
hey! I was setting up the environment all this time. Since it's done now, Can you help me get started with the issue? Like where do i look for the code of the dataview section. Thanks! |
@FabSar You find the all the post-view sections here: https://github.com/ushahidi/platform-client/tree/develop/app/main/posts/views and the data-view more specifically here: https://github.com/ushahidi/platform-client/blob/develop/app/main/posts/views/post-view-data.directive.js. |
okay thanks a lot! |
@FabSar is this issue available, or are you still working? Thanks |
Hi. @FabSar is this issue available, or are you still working? let us know if you need some help. |
I cannot replicate this issue locally, has this been fixed? |
Just opened a PR for this, here: ushahidi/platform-client#1365 |
I am able to replicate this issue locally, @rowasc can i work on this issue? |
@rohit645 Thank you for your interest in contributing, please go ahead and work on this and don't hesitate to reach out if you need further guidance! |
@rohit645 How is it going, are you still working on this? Please let me know if you need any guidance! |
@Angamanga, I am trying to figure it out. I will be in touch shortly. |
@Obadha2 Could you QA in Steve Buscemi?
|
QA'd, passes. 👍 |
Problem
If I delete a post in dataview the total number of posts displayed on the top still remains the same (for example, before deleting: "20 of 20 posts". After deleting: "19 of 20 posts"
Reproduce the bug
Implementation notes
Fix in https://github.com/ushahidi/platform-client repo
Hint 💡 The data-view files can be found here: https://github.com/ushahidi/platform-client/tree/develop/app/main/posts/views
When you are done, 🔀 Start a Pull Request. There are two ways how you can start a pull request:
If you are familiar with the terminal or would like to learn it, here is a great tutorial on how to send a pull request using the terminal.
You can also edit files directly in your browser and open a pull request from there.
🏁 When done Ask in comments for a review :)
Want to work on this issue?🙋
Comment below. If someone else has claimed it, ask if they've opened a pull request already and if they're stuck -- maybe you can help them solve a problem or move it along!
The text was updated successfully, but these errors were encountered: