Skip to content

News item path fix for listing block bug - #8398

Merged
davisagli merged 3 commits into
mainfrom
newsItemPathFix
Aug 5, 2026
Merged

News item path fix for listing block bug#8398
davisagli merged 3 commits into
mainfrom
newsItemPathFix

Conversation

@TimoBroeskamp

@TimoBroeskamp TimoBroeskamp commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

If your pull request closes an open issue, include the exact text below, immediately followed by the issue number. When your pull request gets merged, then that issue will close automatically.

Closes #8397


If your pull request includes changes to the documentation—either in narrative documentation, Storybook, or configuration—then a pull request preview will be generated and a link will populate in the description of your pull request below.
By clicking that link, you can use the visual diff menu in the upper right corner to navigate to pages that have changes, then display the diff by checking the Show diff checkbox.

@TimoBroeskamp
TimoBroeskamp requested review from ericof, jackahl and sneridagh and removed request for ericof August 5, 2026 12:52

@jackahl jackahl left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, I wrote that code. So LGTM

@davisagli davisagli left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This matches what DefaultView does, so should be fine. Thanks!

Comment thread packages/volto/news/8397.bugfix Outdated
@davisagli
davisagli merged commit 260e576 into main Aug 5, 2026
66 of 69 checks passed
@davisagli
davisagli deleted the newsItemPathFix branch August 5, 2026 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Emtpy listing block on a news item loads main page instead of the news item page

3 participants