-
Notifications
You must be signed in to change notification settings - Fork 24
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
Fix/cypress wait #1262
Fix/cypress wait #1262
Conversation
✔️ Deploy Preview for clever-edison-cd22c1 ready! 🔨 Explore the source changes: 1294b08 🔍 Inspect the deploy log: https://app.netlify.com/sites/clever-edison-cd22c1/deploys/60ae1028e4aaf1000875648e 😎 Browse the preview: https://deploy-preview-1262--clever-edison-cd22c1.netlify.app |
LGTM! |
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.
Ditto!
* Put port in the url * Increased node max. memory limit * Change wait-on url * Run backend before build * Run build script from the package.json * Restore some old parts * Updated broken test * Updated changelog Co-authored-by: dominikx96 <dominik@airnauts.com>
This reverts commit 40a63b9.
This reverts commit 40a63b9.
Pull Request Template
Issue
#1257
Addresses # (issue)
Description
I tried to follow all CircleCI steps in my local setup and noticed in the build section in the config.yml command:
yarn build:app:public
- BUT we just set up backend without running it. It means - the build doesn’t contain all the necessary pages which come from the listing so this build is not correct.I moved the build command into sites/public
package.json
, thenyarn start
command should run after build.Also fixed broken
MyApplications
test - it requires to wait for route change after successful login.Type of change
How Can This Be Tested/Reviewed?
Please describe the tests that you ran to verify your changes. Provide instructions so we can review. Please also list any relevant details for your test configuration
Checklist: