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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backport 'Don't show deleted resources in last activities ' to v0.26 #9330

Merged

Conversation

andreslucena
Copy link
Member

馃帺 What? Why?

Backport #9293 to v0.26

鈾ワ笍 Thank you!

* Clarify describe block title

* Improve CSS selector in spec so it's more specific

* Add failing spec

* Make ActionLog aware of deleted resources

* Move the deleted comment factory to a trait

* Fix linter offenses
@andreslucena andreslucena added backport Pull Requests that are a backport for a fixed bug module: comments module: core type: fix PRs that implement a fix for a bug labels May 18, 2022
ahukkanen
ahukkanen previously approved these changes May 19, 2022
@ahukkanen
Copy link
Contributor

@andreslucena There seems to be a broken spec here. I re-ran it but it is still failing due to the same reason.

@andreslucena andreslucena force-pushed the backport/dont-show-deleted-resources-in-9293 branch from ad99108 to a643d99 Compare May 19, 2022 16:21
@andreslucena
Copy link
Member Author

andreslucena commented May 20, 2022

@ahukkanen I've fixed the spec. The problem was related with #8756, as I was using markup that's only available in develop

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Pull Requests that are a backport for a fixed bug module: comments module: core type: fix PRs that implement a fix for a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants