-
-
Notifications
You must be signed in to change notification settings - Fork 91
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
feat: add cypress #386
base: master
Are you sure you want to change the base?
feat: add cypress #386
Conversation
… into feat/add-testing-framework
… into feat/add-testing-framework
… into feat/add-testing-framework
Co-authored-by: V Thulisile Sibanda <[email protected]>%0ACo-authored-by: asyncapi-bot <[email protected]>%0ACo-authored-by: prashantrai-30 <[email protected]>
Co-authored-by: acethecreator <[email protected]>
* initial commit for CFP and speakers form * changes to the form * removed comments * added example env * changes to apply btn * configured the speakers register API * changed path for CFP * url correction * added Online conference in the Navbar * Made CFP banner and details json dependent * update cfp page * Update dates * Update eventDetails.js * test commit * configured to build * temp changes to form * changes to form style * changed email message * changed the email message * changed cfp dates * changes to cfp css * changes to online page * deleted useless components * Update README.md * Update README.md * added speakers guideline component * changes to guidelines * changed carousel image * changes for production * changes to env names * testing changes * test 2 * test 3 * test 4 * test 5 * test 6 * changes to netlify.toml * test 1 * test 2 * test 3 * test 4 --------- Co-authored-by: asyncapi-bot <[email protected]> Co-authored-by: V Thulisile Sibanda <[email protected]> Co-authored-by: Ace <[email protected]> Co-authored-by: acethecreator <[email protected]>
This reverts commit 066c265.
* added online cfp form * testing different toml configuration * ehanced config * testing new config * fixed typo * fixed cfp date
Co-authored-by: Cody's Dad <[email protected]>
✅ Deploy Preview for peaceful-ramanujan-288045 ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
… into feat/add-testing-framework
… into feat/add-testing-framework
…t-coder/conference-website into feat/add-testing-framework
… into feat/add-testing-framework
… into feat/add-testing-framework
… into feat/add-testing-framework
… into feat/add-testing-framework
/ptal |
@AceTheCreator @thulieblack @Mayaleeeee Please take a look at this PR. Thanks! 👋 |
push: | ||
branches: | ||
- master |
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.
I think we only need to run the tests on the incoming changes via a new pull request, so we don't need this part
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.
Sometimes there are direct pushes by the admins, so I guess lets keep it?
- name: Cypress run | ||
uses: cypress-io/github-action@v6 | ||
with: | ||
build: npm run build | ||
start: npm start |
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.
Don't know much about the cypress testing part. Better to follow the official docs and figure it out.
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.
Yeah I did write it from the official docs. Do have a look at them if you want.
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.
I've reviewed it to the best of my knowledge. Let me know what you think @ashmit-coder
Related issue(s)
Resolves #385