Skip to content
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

Restore Unapproved Topics not working as expected #3846 #3856

Merged
merged 1 commit into from Apr 28, 2016

Conversation

@xillibit
Copy link
Member

@xillibit xillibit commented Apr 27, 2016

Pull Request for Issue #3846.

Summary of Changes

Testing Instructions

@xillibit xillibit added this to the 4.0.11 milestone Apr 27, 2016
@rich20
Copy link
Member

@rich20 rich20 commented Apr 27, 2016

  • If you move the complete topic to the trash and restore it, pending messages remain pending messages
  • If you delete only a single pending message - after restore the message is approved
@rich20
Copy link
Member

@rich20 rich20 commented Apr 27, 2016

But I mean, if you delete a single message, you have also checked the message content

@xillibit xillibit force-pushed the xillibit:k4.0_bugs3 branch from d025544 to 30ab688 Apr 27, 2016
@xillibit
Copy link
Member Author

@xillibit xillibit commented Apr 27, 2016

For a single message it's useless because the unapproved state is lost when you delete the message

@rich20
Copy link
Member

@rich20 rich20 commented Apr 27, 2016

  • If you move the complete topic to the trash and restore it, pending messages remain pending messages
  • If you delete only single pending messages (see image) - after restore the messages are approved
    bildschirmfoto vom 2016-04-27 23 23 54
@810 810 merged commit 337fb16 into Kunena:develop Apr 28, 2016
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@xillibit xillibit deleted the xillibit:k4.0_bugs3 branch Feb 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants