test: adding e2e tests - Githubissues

Pull-Request-Community / pull-request-community-website

Pull request community

https://pullrequest.co.il/

MIT License

56 stars 101 forks source link

test: adding e2e tests #249

Closed lirantal closed 2 years ago

lirantal commented 2 years ago

A couple more tests to give folks examples of how to use Playwright.

netlify[bot] commented 2 years ago

Deploy Preview for agitated-hoover-61371e ready!

Built without sensitive environment variables

Name	Link
Latest commit	c1d2f0016a9f52aa97b609620340ea86a7d66c91
Latest deploy log	https://app.netlify.com/sites/agitated-hoover-61371e/deploys/630503c32aa29d00085dd3f4
Deploy Preview	https://deploy-preview-249--agitated-hoover-61371e.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

github-actions[bot] commented 2 years ago

Preview on netlify: https://deploy-preview-249--pull-request.netlify.app/

MichalPorag commented 2 years ago

@lirantal can you please resolve conflicts?

lirantal commented 2 years ago

@MichalPorag done ;-)

MichalPorag commented 2 years ago

@lirantal can you please fix eslint text?

lirantal commented 2 years ago

Same as the other PR

lirantal commented 2 years ago

@MichalPorag ready to merge 🙏🏼

MichalPorag commented 2 years ago

Hi @lirantal, thank you so much! Unfortunately, the PR still has conflicts, can you please fix them?

lirantal commented 2 years ago

Are you sure? looks ok to me

MichalPorag commented 2 years ago

@lirantal yes, I'm sure: https://www.facebook.com/groups/PullRequest/mentorship_application

lirantal commented 2 years ago

?

lirantal commented 2 years ago

@OmriBarZik this one is also mergable if you can review and push it please :-)