-
Notifications
You must be signed in to change notification settings - Fork 27
Qr code component challenge #155
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
base: qr_code_component
Are you sure you want to change the base?
Qr code component challenge #155
Conversation
This reverts commit df8b871.
almirsarajcic
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull from upstream. I recommend reading more about git and GitHub.
|
Noted
ᐧ
…On Mon, Jan 29, 2024 at 11:56 AM Almir Sarajčić ***@***.***> wrote:
***@***.**** requested changes on this pull request.
Pull from upstream. I recommend reading more about git and GitHub.
—
Reply to this email directly, view it on GitHub
<#155 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AXL2DLZIMMHKDG2L5ODJDRLYQ5P2JAVCNFSM6AAAAABCL4VS62VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQNBYGEZDQOBRGQ>
.
You are receiving this because you authored the thread.Message ID:
***@***.***
com>
|
Avoid potential issues with multiple links on the page.
Fix Tailwind CSS config. Improve test to save screenshot for two viewport sizes.
almirsarajcic
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
npm run format
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now the tests are failing.
frontend_mentor_challenges/package.json
Lines 19 to 28 in 7282120
| "scripts": { | |
| "build": "node scripts/build.js", | |
| "check-formatting": "npx prettier -c .", | |
| "format": "npx prettier -w .", | |
| "serve": "npm-run-all --parallel watch simple-autoreload-server", | |
| "simple-autoreload-server": "npx autoreload-server -d ./public/ -p 3000 -b", | |
| "test": "npm-run-all check-formatting build wdio-test", | |
| "watch": "npm run build -- --watch", | |
| "wdio-test": "npx wdio run wdio.conf.js" | |
| } |
Run the
npm run test command locally and fix it until everything is passing.
ae74efd to
50b31a2
Compare
50b31a2 to
2f5528c
Compare
Uh oh!
There was an error while loading. Please reload this page.