Tom Rees-Herdman

Results 525 comments of Tom Rees-Herdman

Note that I've moved this back to the Backlog pending the outcome of this [discussion](https://docs.google.com/document/d/1MGD5Djy6AeeZC4zBtHqS-lQEWD9jw0kf-IIWw-jLCFU/edit?disco=AAABEVx_moQ) on the design doc regarding the shape of the API.

Said discussion is now resolved and this issue is ready for AC.

I've moved this back to AC pending an update once we've confirmed changes to the infrastructure underpinning the "New" badges, as discussed on [Slack](https://10up.slack.com/archives/CBKKQEBR9/p1710423035426139?thread_ts=1710408690.152629&cid=CBKKQEBR9).

Update: I've moved this to the Backlog instead to make it clearer which issues are waiting for a change or approval to the design doc (see [this comment](https://docs.google.com/document/d/1MGD5Djy6AeeZC4zBtHqS-lQEWD9jw0kf-IIWw-jLCFU/edit?disco=AAABJM1aJTE) on the...

The related design doc discussion has been resolved and this is ready for IB.

Hi @ankitrox! Thanks for drafting this IB, nice work so far. A few points: > - [ ] The option will have the following data structure: > ```php > array(...

Thanks @ankitrox, that makes sense. I've made a few amendments to the IB - some points were missed, and a couple of naming and grammatical tweaks were needed. Please review...

Hi @ankitrox, regarding the QAB here - rather than using `fetchExpirableItems()` in the test, I would suggest rewriting it to use (and therefore test) `setExpirableItemTimers()`. Please can you also include...

@eugene-manuilov, #8131 only relates to the UI and logic encapsulated in the [Setup CTA Banner](https://docs.google.com/document/d/1MGD5Djy6AeeZC4zBtHqS-lQEWD9jw0kf-IIWw-jLCFU/edit#heading=h.f4g2vm7vw0r0) component, whereas this issue is about introducing the individual [Audience Tile](https://docs.google.com/document/d/1MGD5Djy6AeeZC4zBtHqS-lQEWD9jw0kf-IIWw-jLCFU/edit#heading=h.smz9mzujems) component, initially in Storybook.

> Thanks, @techanvil, I was confused by what it means `(Storybook)` in the title. So, basically we need to implement the corresponding component, but not connect it to the plugin...