Skip to content

Conversation

@emmadesilva
Copy link
Member

@emmadesilva emmadesilva commented Jul 24, 2024

This PR modifies the BuildService to group all dynamic pages (InMemoryPage instances) together under a "Dynamic Pages" category during the build process. This is because this otherwise can very quickly grow to have a bunch of single item progress bars when using a lot of extended pages of this type.

@codecov
Copy link

codecov bot commented Jul 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (2a97871) to head (cf0bfc8).

Additional details and impacted files
@@             Coverage Diff             @@
##             2.x-dev     #1897   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
- Complexity      1865      1867    +2     
===========================================
  Files            191       191           
  Lines           4923      4927    +4     
===========================================
+ Hits            4923      4927    +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@emmadesilva emmadesilva marked this pull request as ready for review July 24, 2024 14:32
@emmadesilva emmadesilva changed the title Update the build service to group together dynamic pages [2.x] Update the build service to group together dynamic pages Jul 24, 2024
@emmadesilva emmadesilva merged commit a2efff8 into 2.x-dev Jul 24, 2024
@emmadesilva emmadesilva deleted the group-together-dynamic-pages-in-build branch July 24, 2024 14:36
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.

2 participants