jboland88
jboland88
> However, for me and in your screenshot in the testing instructions, it seems as though the list of features is not showing up in the Assistant recommendation Good catch...
@CodeyGuyDylan - updated to fix the react warning you noted 👍
> Also on the survey step, if I have a selected option (currently focused) and press Shift + Tab, the focus goes back to the "Manage Connection" button on the...
> I retested the PR and most things seem to work as expected, but I'm still having the same issue pressing Shift + Tab to go back to the previous...
Thanks @jeherve - I'll add this to our maintenance board to take a look at.
I am moving this to a subtask in the larger My Jetpack roadmap. Going to remove it from the MarTech maintenance board for now
Moving this to icebox for the dotcom roadmap, but leaving it open for now. There may be pieces we'd like to reference going forward.
@chriskmnds - thanks for the review! I've brought back the previous approach and it is indeed much cleaner. It seems the one difference between then and now is an additional...
Thanks again for the review @chriskmnds ! I've moved the logic for setting the visible plans in the comparison grid to its own hook and updated the logic to always...
Thanks again for more feedback @chriskmnds - I have been improved by revisiting and making updates! Indeed, I had become a bit too eager and introduced some self-labeled "improvements" (different...