mcp-playwright icon indicating copy to clipboard operation
mcp-playwright copied to clipboard

add: Optional Bearer Authorization to API POST

Open CopilotMe opened this issue 8 months ago • 2 comments

CopilotMe avatar Mar 19 '25 02:03 CopilotMe

Looks good @CopilotMe , can you please update your branch and send me the PR, we can merge and push it in today's release.

Thanks, Karthik KK

executeautomation avatar Mar 20 '25 21:03 executeautomation

Hi K, If you mean the snapshot conflict I tried a couple of times to update it, but it I seems git does not show any conflicts locally. Please go ahead and just update the branch yourself if you know how to resolve this.

Here is example:

Snapshot Summary
 › 1 snapshot updated from 1 test suite.

Test Suites: 7 passed, 7 total
Tests:       44 passed, 44 total
Snapshots:   1 updated, 1 total
Time:        3.394 s
Ran all test suites matching /#|or|yarn|test/i.
shteryodzhimov@Shteryos-MacBook-Pro mcp-playwright % touch src/__tests__/__snapshots__/tools.test.ts.snap
shteryodzhimov@Shteryos-MacBook-Pro mcp-playwright % git add src/__tests__/__snapshots__/tools.test.ts.snap
shteryodzhimov@Shteryos-MacBook-Pro mcp-playwright % git commit -m "commit snapshot file"
On branch optional-bearer-token
Your branch is up to date with 'origin/optional-bearer-token'.

nothing to commit, working tree clean

CopilotMe avatar Mar 21 '25 01:03 CopilotMe

Yes that looks good to me as well, I will approve it now and merge it !

executeautomation avatar Mar 21 '25 02:03 executeautomation

Can you please remove this file and update the branch, I see there is a conflict

image

Thanks, Karthik KK

executeautomation avatar Mar 21 '25 02:03 executeautomation