Skip to content

Set a default view for Plone Site different by default_view because i…#146

Merged
cekk merged 5 commits intomasterfrom
fix_root_layout
Sep 25, 2025
Merged

Set a default view for Plone Site different by default_view because i…#146
cekk merged 5 commits intomasterfrom
fix_root_layout

Conversation

@cekk
Copy link
Member

@cekk cekk commented Sep 19, 2025

…n plone.restapi 9.15.2, layout is exposed and we don't need it because it broke agid layout

…n plone.restapi 9.15.2, layout is exposed and we don't need it because it broke agid layout
@coveralls
Copy link

Pull Request Test Coverage Report for Build 17920861835

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 70.449%

Totals Coverage Status
Change from base Build 16318855765: 0.0%
Covered Lines: 1397
Relevant Lines: 1983

💛 - Coveralls

@cekk cekk merged commit 21085da into master Sep 25, 2025
13 checks passed
@cekk cekk deleted the fix_root_layout branch September 25, 2025 14:02
mamico added a commit that referenced this pull request Sep 25, 2025
#146)

* Set a default view for Plone Site different by default_view because in plone.restapi 9.15.2, layout is exposed and we don't need it because it broke agid layout

* add view

* test

* zpretty

* test for 3.9, 3.10 removed from gha

---------

Co-authored-by: Mauro Amico <mauro.amico@gmail.com>
cekk added a commit that referenced this pull request Sep 26, 2025
#147)

* Set a default view for Plone Site different by default_view because i… (#146)

* Set a default view for Plone Site different by default_view because in plone.restapi 9.15.2, layout is exposed and we don't need it because it broke agid layout

* add view

* test

* zpretty

* test for 3.9, 3.10 removed from gha

---------

Co-authored-by: Mauro Amico <mauro.amico@gmail.com>

* changelog

* ci disabled

---------

Co-authored-by: Andrea Cecchi <andrea.cecchi85@gmail.com>
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