Skip to content
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

chore(release): Test v7.23.0 #9570

Closed
wants to merge 100 commits into from
Closed

chore(release): Test v7.23.0 #9570

wants to merge 100 commits into from

Conversation

github-actions[bot]
Copy link
Contributor

Production Job Id: 6488057706
Staging Job Id: 6488057673

The basics

Run through this list at least once at staging:

  • Smoke tested the Demo, unauthenticated
  • Created an account
  • Verified invite via mass link works (Team > Invite Button > Invite Link, visit URL in an incognito window)
  • Verified invite via email works. It will show as logs in Datadog.
  • Added Slack, verified meeting notifications (use #t_product_actiontime)
  • Added Mattermost, verified meeting notifications (check the test webhook in Parabol Mattermost)
  • Added GitHub, verified issue created (use ParabolInc/publictestrepo)
  • Added Jira, verified issue created (use parabol-2)
  • Added GitLab, verified issue created
  • Added JiraServer, verified issue created
  • Smoke tested the Retro meeting with 2 players
  • Smoke tested the Sprint Poker meeting with 2 players
  • Smoke tested the Team Check-in meeting with 2 players
  • Smoke tested the Standup meeting with 2 players
  • Smoke tested cards on the dashboard
  • Created a 2nd team
  • Created a 2nd organization
  • Upgraded to Team (Credit card number: 4242 4242 4242 4242, expiration date: any month in the future, CVC: 123)
  • Upgraded to Team with a 3D Secure card (Credit card number: 4000 0000 0000 3220, expiration date: any month in the future, CVC: 123)
  • Tried and failed to upgrade with a card with insufficient funds (Credit card number: 4000 0000 0000 9995, expiration date: any month in the future, CVC: 123). Error feedback is shown in the UI.
  • Smoke tested the app on a mobile device (e.g. navigate between views, smoke test a Retro meeting, etc.)
  • Test previously existed meetings to make sure that existing data is not corrupted
  • Create a task in the discussion thread of a meeting
  • Create a comment in the discussion thread of a meeting

What’s changed

At your discretion, complete the tests for any merged PRs:

  • List each PR with a header and link to ([title], #[issue id])
  • Copy and paste tests, or note what was tested, in this issue
  • Run the equivalent test on staging that you would run in a local environment as closely as possible (e.g. updating the database to simulate cases, etc.)

Added

Fixed

  • admin: fix an issue where ORG_ADMIN cannot see members from team they are not in (#9560) (ef0fbc2)
  • Removed broken Rally links and fixed Youtube links (#9332) (5e98234)

Changed

  • [Snyk] Upgrade core-js from 3.8.1 to 3.36.0 (#9519) (ab47ce4)
  • [Snyk] Upgrade dotenv from 8.0.0 to 8.6.0 (#9494) (1e22931)
  • [Snyk] Upgrade graphql-typed from 0.6.1 to 0.7.2 (#9522) (0ce1384)
  • [Snyk] Upgrade react-dom-confetti from 0.0.10 to 0.2.0 (#9520) (ef68915)
  • [Snyk] Upgrade react-swipeable-views-core from 0.13.1 to 0.14.0 (#9521) (3e42d9b)
  • fix update snyk pr action (#9564) (092e5d9)
  • github: DevOps review if docker folder is modified or release-please-config is changed (#9562) (d18d754)
  • refactor add template mutation to the new sdl pattern (#9533) (fe71841)
  • Roll out AIGeneratedDiscussion to all users (#9554) (b8fa708)

* chore: fix misleading `isLead` field name on `Team`

The field indicates whether the viewer is the lead, but when used in a
query for a different user, the result could be read wrong.

* Fix Team.isLead dependencies
* update error message and increase template limit

* remove max team template limits

* remove canClone prop from CloneTemplate

* remove unused threshold

* remove unused threshold
* feat: Add recurrence to GCal events

* Fun with timezones
…an (#9435)

We ran into timeouts in `getAllProjects`, presumably because we're doing
too many roundtrips. As a quick fix, increse the number of projects
fetched per request from 50 to 500.
Bumps [ip](https://github.com/indutny/node-ip) from 1.1.8 to 1.1.9.
- [Commits](indutny/node-ip@v1.1.8...v1.1.9)

---
updated-dependencies:
- dependency-name: ip
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: parabol-release-bot[bot] <150284312+parabol-release-bot[bot]@users.noreply.github.com>
…documented (#9445)

* Docker compose stack improved

* Remove unused containers from docker-compse and add useful comment on .env.example about PGSSLMODE

* Docker compose profiles added. Documentation extended on how to use the profiles to manage the stack.

* README fixed as docker compose up and down commands were not working

* Typo fixed and docker-compose command replaced by docker compose
* feat: support env-defined saml issuer for PPMIs

Signed-off-by: Matt Krick <[email protected]>

* feat: support single SAML for entire tenant

Signed-off-by: Matt Krick <[email protected]>

---------

Signed-off-by: Matt Krick <[email protected]>
* chore: Associate logs with traces

Add trace information to log output for server side log statements. This
does not include logging from code exclusively used for debugging,
deploying or development.

* Actually add the logger

* Fix DD_LOGS_INJECTION check
Co-authored-by: parabol-release-bot[bot] <150284312+parabol-release-bot[bot]@users.noreply.github.com>
Co-authored-by: parabol-release-bot[bot] <150284312+parabol-release-bot[bot]@users.noreply.github.com>
* feat: add embedder service

---------

Signed-off-by: Matt Krick <[email protected]>
Co-authored-by: Matt Krick <[email protected]>
Co-authored-by: parabol-release-bot[bot] <150284312+parabol-release-bot[bot]@users.noreply.github.com>
Co-authored-by: parabol-release-bot[bot] <150284312+parabol-release-bot[bot]@users.noreply.github.com>
Co-authored-by: parabol-release-bot[bot] <150284312+parabol-release-bot[bot]@users.noreply.github.com>
Signed-off-by: Matt Krick <[email protected]>
Co-authored-by: parabol-release-bot[bot] <150284312+parabol-release-bot[bot]@users.noreply.github.com>
* fix: limit invites from spammers

* update where we check pending emails

* check total plus pending invites

* use invitees instead of pending
Previously we tried to fetch more projects per page, but Jira only ever
returns 50 max. Instead once we know how many projects there are after
fetching the first page, we fetch all remaining pages in parallel.
* fix: replace lone surrogates in draft-js content

Signed-off-by: Matt Krick <[email protected]>

* fix typo

Signed-off-by: Matt Krick <[email protected]>

* fix: eslint errors

Signed-off-by: Matt Krick <[email protected]>

---------

Signed-off-by: Matt Krick <[email protected]>
Bumps [es5-ext](https://github.com/medikoo/es5-ext) from 0.10.62 to 0.10.64.
- [Release notes](https://github.com/medikoo/es5-ext/releases)
- [Changelog](https://github.com/medikoo/es5-ext/blob/main/CHANGELOG.md)
- [Commits](medikoo/es5-ext@v0.10.62...v0.10.64)

---
updated-dependencies:
- dependency-name: es5-ext
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
The following vulnerabilities are fixed with an upgrade:
- https://snyk.io/vuln/SNYK-JS-UNDICI-6252336

Co-authored-by: snyk-bot <[email protected]>
Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.14.8 to 1.15.4.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](follow-redirects/follow-redirects@v1.14.8...v1.15.4)

---
updated-dependencies:
- dependency-name: follow-redirects
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Matt Krick <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Matt Krick <[email protected]>
rafaelromcar-parabol and others added 26 commits March 19, 2024 11:23
Co-authored-by: parabol-release-bot[bot] <150284312+parabol-release-bot[bot]@users.noreply.github.com>
Co-authored-by: parabol-release-bot[bot] <150284312+parabol-release-bot[bot]@users.noreply.github.com>
Co-authored-by: parabol-release-bot[bot] <150284312+parabol-release-bot[bot]@users.noreply.github.com>
@rafaelromcar-parabol
Copy link
Contributor

Failing on Staging

@mattkrick mattkrick deleted the release/v7.23.0 branch October 2, 2024 23:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants