Fast404 prevents css/js assets from generating when aggregation is enabled on Drupal 10.1

This patch is for 2.x, but the problem also exist in 3.x

Since by default, the assets folder is in 'sites/default/files', we don't want to skip everything inside of it, added a suffix 'js' and 'css' to solve this.

Thanks YahyaAlHamad, I was having the same issue. As you mentioned this issue is there for 3.x and I am using 3.x version, so just rerolled it against 3.x version.

Missed the use Drupal\Core\StreamWrapper\AssetsStream; statement in the last one, fixed it.

I was running into this issue and have applied the patch mentioned in #7. It applies smoothly. Thanks for the patch :)

I created the issue fork, I applied the fast_404_css_js_aggregation-3373845-7 patch. I also tested the solution, and found that using fast_404 and aggregation enabled, both js and css tablets are being generated.

Assigning to myself to see if I can get this one in soon.

uploaded to have stable patch for testing

This is working great. Thanks for the patch.

Thanks, everyone, for the great work. I see there are patches for both 2.x and 3.x branches so I'll get both of these in. There are some minor formatting and wording nitpicks, but I will fix these myself before committing.

@siddharthjain_7998 In the future, please don't add a patch that's already been added. You didn't say you tested the patch at all... just that it "applied smoothly". If you are going to check an issue, then it is best if you test it and explain how you tested it.

@jvbrian In the future, there is no need to create a MR if there is a patch and you aren't making any changes. Thank you for explaining that you tested it.

@ikit-claw Fyi, it probably wasn't clear but the MR was just the same as #7, so there was already a patch available.

I've started a test on #7 with 10.1 since that wasn't there before and it failed in #6.

Passed on 10.1:

https://www.drupal.org/pift-ci-job/2740780 →

and this has been tested on 10.1 and the fix targets 10.1+ so this looks safe to merge.

This has been merged for both 2.x and 3.x and will be in the next 3.x release though I don't think there will be any more 2.x releases. I did make some wording and formatting changes.

Automatically closed - issue fixed for 2 weeks with no activity.

To feed the search engines: This is related to Change Record https://www.drupal.org/node/3301716 →