Skip to content

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

Merged
ahukkanen merged 2 commits intorelease/0.26-stablefrom
backport/dont-show-deleted-resources-in-9293
May 24, 2022
Merged

Backport 'Don't show deleted resources in last activities ' to v0.26#9330
ahukkanen merged 2 commits intorelease/0.26-stablefrom
backport/dont-show-deleted-resources-in-9293

Conversation

@andreslucena
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

@andreslucena andreslucena requested a review from ahukkanen May 20, 2022 10:08
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.

3 participants