Skip to content

List nodes in default group#404

Merged
robertcheramy merged 3 commits into
ytti:masterfrom
oogali:browse-default-group
Dec 3, 2025
Merged

List nodes in default group#404
robertcheramy merged 3 commits into
ytti:masterfrom
oogali:browse-default-group

Conversation

@oogali

@oogali oogali commented Nov 28, 2025

Copy link
Copy Markdown
Contributor

When the list of nodes is rendered, a link is created for the group.

However, if a node doesn't have a group it's rendered as 'default', but there isn't a way to actually link to the default group.

Pre-Request Checklist

  • Passes rubocop code analysis (try rubocop --auto-correct)
  • Tests added or adapted (try rake test)
  • Changes are reflected in the documentation
  • User-visible changes appended to CHANGELOG.md

Description

This PR adds logic to allow a client to select the default group (which isn't actually a group, but a nil value).

@oogali oogali changed the title Add condition to match a nil group List nodes in default group Nov 28, 2025
Comment thread lib/oxidized/web/webapp.rb Outdated
When the list of nodes is generated, a link is created for the group.

However, if a node doesn't have a group it's rendered as 'default', but
there is no way to actually link to the default group without this new
conditional.
@oogali oogali force-pushed the browse-default-group branch from 0a5cb03 to 01ae22a Compare November 29, 2025 15:32
@robertcheramy robertcheramy merged commit f2c1dd2 into ytti:master Dec 3, 2025
7 checks passed
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.

2 participants