Skip to content

Conversation

@cquirosj
Copy link
Contributor

@cquirosj cquirosj commented Jan 5, 2026

Backporting to release-6.8 branch fix for:

DharmishthaVPatel and others added 2 commits January 5, 2026 11:44
…-fix

Point RavenDB Management Studio link to localhost
# Fix RavenDB Management Studio link binding issue

## Summary
This PR fixes a property name mismatch introduced in PR #5237 that caused the RavenDB Management Studio hyperlink to be grayed out and non-functional in the ServiceControl Management UI.

## Problem
After merging PR #5237 "Point RavenDB Management Studio link to localhost", the RavenDB Management Studio link in the Advanced Options maintenance mode section became unresponsive. The link appeared grayed out and clicking it had no effect.

## Root Cause
PR #5237 renamed the property from `StorageUrl` to `RavenDbStudioUrl` in `ServiceControlBaseService.cs`, but the corresponding ViewModel property in `ServiceControlAdvancedViewModel.cs` was incorrectly named `RavenStudioUrl` (missing "Db"). This property name mismatch caused the WPF data binding to fail.

## Solution
- Fixed property name in `ServiceControlAdvancedViewModel.cs` from `RavenStudioUrl` to `RavenDbStudioUrl`
- This ensures proper binding between the ViewModel and the underlying ServiceControlInstance property

## Files Changed
- `src/ServiceControl.Config/UI/AdvancedOptions/ServiceControlAdvancedViewModel.cs`

## Testing
- RavenDB Management Studio hyperlink now appears as an active blue link
- Clicking the link properly opens the RavenDB Studio at `http://localhost:{port}/studio/index.html#databases`
- Right-click context menu "Copy to Clipboard" functionality works correctly

Fixes issue introduced in #5237
@cquirosj cquirosj changed the base branch from master to release-6.8 January 5, 2026 17:48
@cquirosj cquirosj changed the title [Backport to 6.8] Fix for management utility hostname fields validation rejects hyphens and allows invalid characters [Backport to 6.8] Fix for RavenDB Studio cannot be reached using a custom hostname Jan 5, 2026
@cquirosj cquirosj merged commit 147f2e1 into release-6.8 Jan 5, 2026
32 checks passed
@cquirosj cquirosj deleted the backport-fix-for-2642 branch January 5, 2026 18:03
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.

3 participants