-
Notifications
You must be signed in to change notification settings - Fork 46
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
upgraded cypress and debugged mapTools.js #102
Conversation
@vishal5251 thanks for you work! bty, are you in the slack channel? |
nope.. |
// with the any changed environment variables | ||
return config | ||
} | ||
module.exports = (on, config) => {}; |
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.
@vishal5251 can you double check this change, the line require('@cypress/react/plugins/react-scripts')(on, config)
is for the cypress component tool, after deleting this line, does the component tool works? https://github.com/Greenstand/treetracker-wallet-web#how-to-build-a-component
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.
@dadiorchen Yes, I commented the line and it still it works.
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.
Good, thank you for you contribution.
We have a community on slack, if you like you can leave you email here or send it to me so we can invite you to join the community! |
[email protected] this is my gmail account. |
I am not much familiar with Cypress. @vishal5251 Do you know how to fix this issue? |
@pvdo did you try to restore these lines:
|
That actually works! I should have done that 😑 |
I had upgraded the cypress version and debugged the mapTool.js (path: C:\Users\Vishal\Documents\GitHub\treetracker-wallet-web\src_old\mapTools.js).