Skip to content

refactor: drop the Pinned section from spaces#448

Merged
tkuhn merged 1 commit intomasterfrom
remove-pinned-section-from-spaces
Apr 22, 2026
Merged

refactor: drop the Pinned section from spaces#448
tkuhn merged 1 commit intomasterfrom
remove-pinned-section-from-spaces

Conversation

@tkuhn
Copy link
Copy Markdown
Contributor

@tkuhn tkuhn commented Apr 22, 2026

Summary

  • Pinned resources are now surfaced via views, so the separate `pinned-section` in `SpacePage` is redundant.
  • Removes the section from `SpacePage` (both the data-initialized and lazy-load branches) and its sole consumer `PinGroupList` (Java + HTML).

Test plan

  • Load a space page and confirm views render as before, without the trailing Pinned block.
  • Confirm nothing else in the repo references `PinGroupList` or the `pinned-section` wicket id.

🤖 Generated with Claude Code

Pinned resources are now surfaced via views, so the separate
pinned-section in SpacePage is redundant. Removes the section and its
sole consumer PinGroupList.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@tkuhn tkuhn merged commit c9b9c0d into master Apr 22, 2026
7 checks passed
@tkuhn tkuhn deleted the remove-pinned-section-from-spaces branch April 22, 2026 14:30
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.

1 participant