fluentui-blazor icon indicating copy to clipboard operation
fluentui-blazor copied to clipboard

fix: DocViewer doesn't support multi level include

Open AClerbois opened this issue 11 months ago • 1 comments

🐛 Bug Report

I tried to include a document into another document where this document is also included.

💻 Repro or Code Sample

Branch : dev-v5

My operations were :

  • In the project, FluentUI.Demo.Client, I create a document NewA.md into the existing document MigrationColor.md. This one is already included into the MigrationVersion5.md

While the visionalisation, I get an error message : "The Component INCLUDE error"

If I include the NewA.md in the MigrationVersion5.md, the content is display.

🤔 Expected Behavior

Display the included file

😯 Current Behavior

Display an error

💁 Possible Solution

I put the document one level upper.

🔦 Context

In the branch dev-v5

🌍 Your Environment

  • OS & Device: Windows 11, Microsoft Laptop Studio 2
  • Browser : Microsoft Edge, Google Chrome, Mozilla FireFox
  • .NET and Fluent UI Blazor library Version : dev-v5

AClerbois avatar Jan 19 '25 08:01 AClerbois

I know that recursive includes are not possible. It is not yet a necessary function. Especially since to do that you need to detect A->B->A include loops. And that's another story.

But all new PRs are welcome 😀

dvoituron avatar Jan 19 '25 10:01 dvoituron