Skip to content

Conversation

@maccabeelevine
Copy link
Member

@maccabeelevine maccabeelevine commented Dec 22, 2025

Abstract server URL generation away from the underlying Laminas view helper, as suggested by @demiankatz :

Maybe a good strategy would be to build a top-level service somewhere to get server URLs. The initial implementation could be a thin wrapper around the view helper, but that would result in a cleaner overall design, and might make it easier to swap out a different implementation in the future if we find ourselves needing to step back from the view helper approach.

And extracted from #4954 as also suggested

Copy link
Member

@demiankatz demiankatz left a comment

Choose a reason for hiding this comment

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

Thanks, @maccabeelevine!

@demiankatz demiankatz added this to the 11.1 milestone Dec 23, 2025
@demiankatz demiankatz merged commit 2a9a5a1 into vufind-org:dev Dec 23, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants