-
Notifications
You must be signed in to change notification settings - Fork 43
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add playwright script to take screenshots #37
Conversation
Love how concise this is, though it does now introduce managing a separate Python environment now. Can we keep it this simple in JS? |
Welcome to new-connect! Make sure to:
deployed preview: https://37.connect-d5y.pages.dev |
isn't just links to the screenshot better? The PR looks cluttered |
This change doesn't post them to the PR comments, I just added them here to show the screenshots that the script produces. But yeah we could post links, or just make them smaller in the comment. Working on that in #38 |
Got this even after
|
Ah yeah need to install the browser |
Takes screenshots of the login page, demo
RouteList
, andRouteActivity
.To run:
Also made a follow up to add them to the PR comments: #38
For #36