Ben Bowler

window is not defined

13

I'm not able to load the plugin at all via pnpm. ``` window is not defined ReferenceError: window is not defined at Object. (../..//node_modules/.registry.npmjs.org/suggestions/1.6.0/node_modules/suggestions/index.js:57:1) at Module._compile (module.js:652:30) at Object.Module._extensions..js (module.js:663:10)...

If you delete an attached post it shows an error on the parent post. I've solved the issue using hooks, code below, let me know where to add this into...

Add documentation

1

I've added some documentation that describes how to generate and prepare the certificates for adding to the GitHub Actions Secrets. I you have any suggestions let me know.

Create Audience Tile Error component.

3

## Summary Addresses issue: - #8228 ## Relevant technical choices See comment below. ## PR Author Checklist - [x] My code is tested and passes existing unit tests. - [...

Add site health checks for Ads module.

1

## Summary Addresses issue: - #8245 ## Relevant technical choices In order to implement `ModuleWithTag` I had to implement a `register_tag` function: https://github.com/google/site-kit-wp/blob/9a2dfa37f626b6ed9f96043a0b2b8cf4763bad58/includes/Modules/Ads.php#L132-L135 I also tested the regex used would...

Missing required key 'FunctionName' in params

1

> Running "lambda_deploy:prod" (lambda_deploy) task > > > AWS API request failed with undefined - MissingRequiredParameter: Missing required key 'FunctionName' in params I've set the function option but I can't...

We have the following config but we get a no fast forwards error when running the command which suggests the flag is not being applied. ``` gitpush: { stage: {...

Artist and Release Fields

3

There currently aren't artist or release fields in the system. Was this deliberate? If I added support of these fields would you accept a pull request?

Add reusable Widget area, New badge feature.

3

## Summary Addresses issue: - #8169 ## Relevant technical choices Implemented as in IB. ## PR Author Checklist - [ ] My code is tested and passes existing unit tests....

Refactor the Audience Tiles to use new pivot report infrastructure

1

## Feature Description As discussed on [Slack](https://10up.slack.com/archives/CBKKQEBR9/p1712306329670989), while we are initially implementing the Audience Tiles to use separate, per-audience reports to retrieve the cities and "top content" metrics (one report...