-
Notifications
You must be signed in to change notification settings - Fork 35
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
Deploy to Production – 2023-11-28 #2415
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
# Conflicts: # site/gatsby-site/cypress/e2e/integration/discover.cy.js
* Add Cypress e2e tests as Github action * Add folder path to `cypress.yml` Github action * Add npm install step * Move to the site directory * Set working directory * Use npm install instead of yarn * add --legacy-peer-deps * `install: false` on cypress action * add github token * remove install-command from cypress step * Print Cypress record key * Increase query timeout to 15000 ms * Move checkout step to the top * Add custom install-command * Fix Cypress config file name * Add wait-on: http://localhost:8000/ * Add npm start and increase timeout to 2 mins * Add ENV vars from Github secrets * Remove unused npm install step * Add projectId to Cypress * Add missing env variables on Cypress tests * Increase query timeout from 15000 to 30000 * Record tests on Cypress dashboard * Add parallel tests * Add 10 parallel jobs * Split install and tests * Add working-directory * Add `install` job as initial job * Upload and download artifacts * Add cache to node_modules * Remove dependency * Touch to trigger deploy * Add cache to install job * Add npm list to check npm modules * Add echo for cache hit * Remove step IF condition * Fix typo on step syntax * Use relative path to cache node_modules * Cache `~/.cache/Cypress` folder * Change `npm install` to `npm ci` * Change Cypress group name * Start the Gatsby server in a separate job * Add env vars to `npm start` * Add wait-on on `npm start` * Add `RUNNER_TRACKING_ID="" && ` * Change `npm start` to `npm serve` * Add `run` on `npm` scripts * Change Cypress port to `9000` * Fix cache folder paths * Upload and download `public` and `.cache` folders as artifacts * Change upload/download artifact to cache * Add jobs' name * Remove unused `serve` script from `package.json` * Add Github secrets documentation in the README file * Remove timeout from `query` command * Change `wait-on-timeout` to 5mins * Remove `netlify.toml` file. Netlify won't run the Cypress tests anymore. * Remove just the `netlify-plugin-cypress` plugin from `netlify.toml` * Comment `.cache` and `public` folder cache * Add 10 mins timeout for `gatsby develop` * Rename job accordingly * Increase temporary `defaultCommandTimeout` to 1 min * Add `GATSBY_REALM_APP_GRAPHQL_URL` new env variable since Realm GraphQL service url changed * Add missing `GATSBY_REALM_APP_GRAPHQL_URL` variable to `cypress.yml` job * Fix typo on `cypress.yml` * Reduce `defaultCommandTimeout` to 30s * Touch README * touch README * Add `Deploy to Netlify` job * Increase temporary `defaultCommandTimeout` to 1 min * Comment job dependency to test Netlify deploy * Add `npm run build` * Add `--legacy-peer-deps` flag * Separate `npm install` and `npm run build` * change `netlify.toml` file path * change `netlify.toml` file path * Change `publish-dir` path * Set deploy timeout to 10 mins * Remove `Deploy to Netlify` job from GH action * Set the commit message to the pull request title * Add more info on a comment * Trigger deploy * Only process videos in the case that tests fail (`videoUploadOnPasses`: `false`) * Add `MONGODB_MIGRATIONS_CONNECTION_STRING` to cypress step * Tag cypress record with branch name * Add `https://pablo-staging-aiid.netlify.app` to Realm allowed origins * Add `npm run build` step * add `env` variables to `npm run build` step * cache `public` folder * Remove unused step id * Set `commit` echo output * Use `gatsby serve` * Change serve port to use `8000` * Cypress to wait for port 8000 * Change `public` folder cache name * Increase parallel machines from 5 to 10 * Undo unnecessary graphql api url change * only run actions against specific branches * Move cypress workflow into reusable one * Fix test name * Make realm staging deploy reusable * Enable cypress videos * Add test workflow * tmp delete netlify config * undo tests * cleanup diff * moar cleanup --------- Co-authored-by: Pablo Costa <[email protected]>
✅ Deploy Preview for staging-aiid ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Fix/text editor bullets
Add roles documentation
Change `editor` role to `incident_editor`
* Add onResendConfirmationClick function on the toast message * Display resend option only on 'confirmation required' error * Add tests to the confirmation email * Move discover.cy.js file to test * Revert "Move discover.cy.js file to test" This reverts commit b713802. * Improve flakiness on discover test
* Stay on current page when claiming/unclaiming - Stays on current page when claiming or unclaiming - Adds unclaim functionality * check for incident editors before filtering * disableSortBy for submitters and editors * Save table state to avoid refreshing filters on claiming * Add claims a submission test * Add claim/unclaim tests * Add more tests for page validation * Fix tests to use logged in user information
* Fallback to sponsors JSON if no sponsors in prismic * Fix sponsors modal with Prismic * Use React markdown to display sponsor text * Fix test for sponros
* Uses prismic to display docs (about, etc) * Add doc prismic type * Enable mdx fetch besides prismic docs * Use only one page creator for docs * Add test for create doc pages
* Remove duplicated query and limit users to 9999 * Trigger deploy * Fix map of editors * Fix admin test to use mock * Add missing users to. mock * Fix users list * Fix user selection on userfields dropdown
kepae
approved these changes
Nov 27, 2023
mega nitpick: recurs -> recurses
…st-parsing Fetch articles from sites using cookies
Fix/discover missing translations
@kepae should be safe to merge |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Including:
editor
role toincident_editor
#2401