Skip to content

Conversation

@ighristov
Copy link
Contributor

@ighristov ighristov commented Sep 12, 2022

Added info for Blazor Native Report Viewer, changed a bit the existing paragraph, explaining the old Blazor Viewer internally uses HTML5 Report Viewer.

Note to external contributors: make sure to sign our Contribution License Agreement (CLA) for Blazor UI first:

https://forms.office.com/Pages/ResponsePage.aspx?id=Z2om2-DLJk2uGtBYH-A1NbWxVqugKN5DvVp8I-1AgOBURFBVSkwyMlA1TkFDVFdMNU1aM1o1UlZQOC4u

closes https://github.com/telerik/blazor/issues/4911

Added info for Blazor Native Report Viewer, changed a bit the existing paragraph, explaining the old Blazor Viewer internally uses HTML5 Report Viewer.
@ighristov ighristov requested a review from a team September 12, 2022 10:36
@dimodi
Copy link
Contributor

dimodi commented Sep 13, 2022

@ighristov Thanks, Ivan! I have two questions -

  1. Is this URL still valid? I see it works, but there is a redirect. Should we link to the new one instead?
  2. I split the long paragraph about the older Report Viewer to a bullet list, for better readability. Currently it looks like the old viewer has more features than the new one. If this is not true, perhaps we should tweak the content to avoid confusion?

@ighristov
Copy link
Contributor Author

@dimodi
Hi Dimo!

  1. Yes, it's still valid - we'll keep the both Blazor Report Viewers available for some time - at least until the Native Blazor viewer is fully implemented. You are right, our latest documentation refactoring introduced this redirect, so the new URL should be used instead: https://docs.telerik.com/reporting/embedding-reports/display-reports-in-applications/web-application/blazor-report-viewer/overview
  2. Correct - the old viewer has more features than the new one and this is reflected in our docs as well. Our goal is to close the features gap in the following releases and make the new viewer on par with the old one.

@dimodi dimodi merged commit 4f63cec into telerik:master Sep 13, 2022
@ighristov ighristov deleted the patch-1 branch September 13, 2022 11:11
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