Skip to content

Fix broken links to ReadTrace reports since file names have been changed#486

Merged
PiJoCoder merged 1 commit intomasterfrom
FixReadTraceReportNotFound_pijocoder_032526
Mar 26, 2026
Merged

Fix broken links to ReadTrace reports since file names have been changed#486
PiJoCoder merged 1 commit intomasterfrom
FixReadTraceReportNotFound_pijocoder_032526

Conversation

@PiJoCoder
Copy link
Collaborator

To test:

  1. Open a ReadTrace report
  2. Test all links to other reports inside all reports (not too many)

…with SQL Nexus prebuild process and naming conventions
@PiJoCoder
Copy link
Collaborator Author

related to #478

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates ReadTrace report drillthrough targets to match the renamed *_C report filenames so report-to-report navigation works again across the packaged (sqlnexus/Reports/*.rdlC) and source (NexusReports/*.rdl) report sets.

Changes:

  • Update multiple ReadTrace reports’ <Drillthrough><ReportName> values to reference the renamed *_C report names.
  • Add ReadTrace reports to NexusReports.rptproj so they’re included in the SSRS project.
  • Minor reordering of ReadTrace_Main_C.rdlC content inclusion in sqlnexus.csproj.

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
sqlnexus/sqlnexus.csproj Reorders/ensures ReadTrace_Main_C.rdlC is included as content copied to output.
sqlnexus/Reports/ReadTrace_UniqueStmtTopN_C.rdlC Fixes drillthrough target to ReadTrace_UniqueStmtDetails_C.
sqlnexus/Reports/ReadTrace_UniqueBatchTopN_C.rdlC Fixes drillthrough target to ReadTrace_UniqueBatchDetails_C.
sqlnexus/Reports/ReadTrace_UniqueBatchDetails_C.rdlC Fixes drillthrough target to ReadTrace_UniqueStmtDetails_C.
sqlnexus/Reports/ReadTrace_Main_C.rdlC Fixes multiple drillthrough targets to *_C report names.
sqlnexus/Reports/ReadTrace_InterestingEvents_C.rdlC Updates drillthrough targets to *_C report names (including self-drillthrough).
sqlnexus/Reports/ReadTrace_Grouping_C.rdlC Updates drillthrough targets to *_C report names.
NexusReports/ReadTrace_UniqueStmtTopN_C.rdl Mirrors drillthrough fix to ReadTrace_UniqueStmtDetails_C in source RDL.
NexusReports/ReadTrace_UniqueBatchTopN_C.rdl Mirrors drillthrough fix to ReadTrace_UniqueBatchDetails_C in source RDL.
NexusReports/ReadTrace_UniqueBatchDetails_C.rdl Mirrors drillthrough fix to ReadTrace_UniqueStmtDetails_C in source RDL.
NexusReports/ReadTrace_Main_C.rdl Mirrors multiple drillthrough target updates to *_C report names in source RDL.
NexusReports/ReadTrace_InterestingEvents_C.rdl Mirrors drillthrough target updates to *_C report names in source RDL.
NexusReports/ReadTrace_Grouping_C.rdl Mirrors drillthrough target updates to *_C report names in source RDL.
NexusReports/NexusReports.rptproj Adds ReadTrace reports to the SSRS project’s report list.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@PiJoCoder PiJoCoder merged commit 13eb811 into master Mar 26, 2026
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.

3 participants