Tom Rees-Herdman

Results 525 comments of Tom Rees-Herdman

Update vulnerable NPM dependencies with auto-fixable version updates

Hey @ivonac4, it's blocked by #5862 which is still in IB, we need to execute that issue to move this one forward.

Update vulnerable NPM dependencies with auto-fixable version updates

@binnieshah yes, it's still blocked by #5862 as per the comment above.

Update the help menu’s list of items.

Thanks @abdelmalekkkkk, great spot - that has simply been missed. I've asked @sigal-teller about it on [Figma](https://www.figma.com/design/SquElvNorSTJOTV1I7oYAq?node-id=3657-21829#1538944480). We can proceed with this issue as defined, and open a new one...

Update the help menu’s list of items.

Thanks @abdelmalekkkkk! That LGTM. IB ✅

Show the Welcome modal on the dashboard.

Thanks @abdelmalekkkkk, this IB is looking good. A couple of small points, I think we should: - Put the feature flag condition for rendering the modal in `DashboardMainApp` rather than...

Show the Welcome modal on the dashboard.

Thanks @abdelmalekkkkk, LGTM! IB ✅

Show the Welcome modal on the dashboard.

Hey @abdelmalekkkkk, I've just noticed that the IB doesn't cover this part of the AC. Please can you update it accordingly? > - For the dashboard tour variant, a tooltip...

Patch React Joyride to support tooltip target.

IB ✅

Patch React Joyride to support tooltip target.

Hey @mohitwp, good spot! This seems to be a glitch that's only occurring in Storybook when the story is embedded in an iframe. If you open the story in a...

Help menu not present on Analytics and Key Metrics setup screens

Hey @abdelmalekkkkk, this IB LGTM. With regard to the estimate, this looks like a "legit" 3, which means it should be bumped to a 7 according to our "bump to...