Skip to content

Fix for 1248 - Corrected handling of section layout FlowLayout/Vertical in provisioning - #1271

Open
Magnus-K-work wants to merge 2 commits into
pnp:devfrom
Magnus-K-work:bugfix/1248-fix-for-flowlayout-in-template-provisioning
Open

Fix for 1248 - Corrected handling of section layout FlowLayout/Vertical in provisioning#1271
Magnus-K-work wants to merge 2 commits into
pnp:devfrom
Magnus-K-work:bugfix/1248-fix-for-flowlayout-in-template-provisioning

Conversation

@Magnus-K-work

Copy link
Copy Markdown

Related issue:
#1248 - Template provisioning can't handle section layouts or published state?

What is changed:

  • ClientSidePageContentsHelper.cs - lines 269-273 Uses correct layout type for FlexibleLayoutSection and FlexibleLayoutVerticalSection
  • ClientSidePageContentsHelper.cs - line 203, re-throw correctly to keep stacktrace and inner exceptions
  • TemplateListInstancesWriter.cs - line 127-129, handle null value for xmlField.Attribute("Required"), as this was causing null point exceptions in test runs

Tests:

  • Not run in full, as many provisioning tests require installation of O365 Starter Intranet which is outdated/archived

Second public PR ever, please help me improve. :)

BR
Magnus

@Magnus-K-work

Copy link
Copy Markdown
Author

I'll keep these PRs alive by merging up from PnP.Framework / Dev for each release.

@Adam-it Adam-it added the area: provisioning ⚙ Provisioning engine issue or pull request label Aug 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: provisioning ⚙ Provisioning engine issue or pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants