Automattic / Automattic/nginx-http-concat

Add support for conditional extras

Open
#13 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
PHP
Stars
120
Forks
37
PR merge metrics
No merged PRs in 30d

Description

We don't support the conditional at the moment for concatenated scripts.

Conditionally loaded scripts should be excluded from concatenation.

Related core code: https://core.trac.wordpress.org/browser/trunk/src/wp-includes/class.wp-scripts.php#L269

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with the linked WordPress class.wp-scripts.php entry point around line 269, then inspect this plugin's concatenation path to see how conditional scripts are handled. Done means conditionally loaded scripts are excluded from concatenated resources; the issue mentions no test file, so identify the relevant existing test or verification path first.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
backend
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.