Paul Pettengill

Results 22 comments of Paul Pettengill

Not sure what the PR process is here. Looks like this gem hasn't required an update in the last couple of years. cc: @RobinDaugherty @banister

**Bump** @RobinDaugherty @banister

I also noticed that pasting strings with more than 10 characters into phone number field causes an error: Uncaught TypeError: Cannot read property '0' of null jquery.formance.min.js:1 reFormatPhoneNumber

Note: If I manually hit the URL below, for the version when I specified it (121.0.6167.85). It returns 200 for me. It appears that is what your script should resolve...

Just for fun, I tried manually specifying the last stable v120 version 120.0.6099.109. ````yaml - browser-tools/install-chrome: chrome-version: 120.0.6099.109 replace-existing: true - browser-tools/install-chromedriver ```` It installed Chrome fine, but did not...

@arthurthefourth That totally makes sense for using the latest. Still not sure why specifying previous versions with supported chromedrivers is breaking though.

I missed that initially. My guess would be that since `jq` is not installed by default, without a separate installation step, you probably get this result. If it's a prereq...

So looking into the PR history, it looks like `jq` was recently added to help out with the fallback condition of getting a 404 in the first place. I'm guessing...

It's broken for me again. ```` Chrome version major is 121 Installed version of Google Chrome is 121.0.6167.85 404 Matching Chrome Driver Version 404'd, falling back to first matching major...

Yes please! I was just getting ready to create this same PR for the same reason.