search

deco-cx / apps

App definitions by deco.cx
Apache License 2.0
82 stars 85 forks source link

Sites with PageInclude break during apps update due tu duplicated section ID (partials) #854

Closed matheusgr closed 2 months ago

matheusgr commented 2 months ago

Issue Type

Please inform the type(s) of issue(s) you are reporting:

Description

Tested on Apps: 0.59.2

Steps to Reproduce (for bugs)

  1. Include a global section on app
  2. Include a PageInclude on a page

Expected Behavior

Page should be rendered (section globals should appear both on the main page, and on PageIncluded).

Actual Behavior

Site breaks due to duplicated section ID on globals.

matheusgr commented 2 months ago

https://github.com/deco-cx/apps/pull/855