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.13.3 #9366

Merged
merged 111 commits into from
Jan 18, 2024
Merged

chore(release): Test v7.13.3 #9366

merged 111 commits into from
Jan 18, 2024

Conversation

github-actions[bot]
Copy link
Contributor

Production Job Id: 5963231530
Staging Job Id: 5963231500

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 3063, 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.)

Fixed

Dschoordsch and others added 30 commits November 13, 2023 11:06
* chore: Users with noTemplateLimit flag can use paid templates

* Don't show a lock if a user can access premium templates

regardless whether it's because their tier or a feature flag
* chore: Add SAML setup to new checkout flow

* Add Authorization also to mobile dash
Co-authored-by: parabol-release-bot[bot] <150284312+parabol-release-bot[bot]@users.noreply.github.com>
* feat: add GCS as a FileStore

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* self-review: fix comments

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* fix: checkExists

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* dockerize testing the branch

* add recent filestore changes to gcs

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* throw on trailing slash

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* retry GCS flakey pushes

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* dockerize

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* bump yarn.lock

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* fix yarn

Signed-off-by: Matt Krick <matt.krick@gmail.com>

---------

Signed-off-by: Matt Krick <matt.krick@gmail.com>
Co-authored-by: Rafael Romero <rafael@parabol.co>
Signed-off-by: Matt Krick <matt.krick@gmail.com>
…9162)

* chore: Users with noTemplateLimit flag can create custom templates

* Allow creating templates on legacy dialog with noTemplateLimit flag
* chore: test case: should not run for no migration

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* test: working case

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* test: failing case

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* debug failing case

* test basename

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* test working case

* better logs

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* test no migration

Signed-off-by: Matt Krick <matt.krick@gmail.com>

---------

Signed-off-by: Matt Krick <matt.krick@gmail.com>
* fix: Avoid caching permissions with different arguments

Permissions are cached by function name. When these depend on the
arguments passed, they should have their own cache key, see
https://the-guild.dev/graphql/shield/docs/rules#limitations

* Make isViewerOnTeam strict

The passed function might access source or args and thus we should use
the strict cache.
* fix: sort team names alphabetically

* sort allTeamsOnOrg
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>
* Sorted .env.example and added more documentation

* Added changes related to PR. Improved section naming.

* Fixed CI variable location

* Sorted variables and created new categories for GOOGLE, DATADOG, AWS. Added missing variables.

* Requested changes incorporated

---------

Co-authored-by: Rafael Romero <rafael@parabol.co>
* feat(trials): Start Trial mutation

* GQL + FE updates

* Copy update

* Minor fixes

* End Trial mutation

* Some cleanup

* More fixes

* Rename var

* Extract 'getFeatureTier'

* More cleanup

* Tiny cleanup

* CR: change to featureTier and billingTier

* CR: Misc fixes

* Rename featureTier back to just tier

* CR: Explicitly do OrgUser and User updates in order

* CR: Combine rethink queries

* Actually we don't need to do those in a specific order

* Why are tests broken

* Revert "CR: Combine rethink queries"

This reverts commit 04ad78d.

* Revert "Why are tests broken"

This reverts commit eca6417.

* Manually bisecting changes that break tests (part 1)

* Revert "Manually bisecting changes that break tests (part 1)"

This reverts commit db90ead.

* Revert "Revert "CR: Combine rethink queries""

This reverts commit a8c21a7.

* Fix tests 🤦

* Update trial copy for OrgPlansAndBillingHeading
* feat: Add Microsoft login

* Microsoft login button styling

* Add Microsoft login to forgot email and team invitation dialogs

* Cleanup and pseudoId

* Formatting

* Use colors from the palette for oauth login buttons

* Cleanup
* chore(kudos): add kudos team settings

* store emoji id
* chore(kudos): add kudos team settings

* store emoji id

* chore(kudos): add kudos record when adding emoji reaction

* Fix snackbar

* quick fix for types

* Dynamic emoji

* del console log

* Remove comment

* Remove unused kudos field

* expose senderUser

* Fix reactable type

* Fix table name

* Don't check feature flag

* Fix analytics

* Add snackbar events

* fix typescript
* chore(kudos): add kudos team settings

* store emoji id

* chore(kudos): add kudos record when adding emoji reaction

* Fix snackbar

* quick fix for types

* Dynamic emoji

* del console log

* feat(kudos): display notification when kudos received

* Remove comment

* Update notification text

* Remove unused kudos field

* expose senderUser

* Fix reactable type

* Fix table name

* remove unused fields

* Don't check feature flag

* Fix analytics

* Add snackbar events

* fix typescript

* Fix type

* Fix type

* add notification status analytics

* Fix type

* fix type

* Fix table name caused by wrong merge

* add snackbar clicked event to kudosReceived snackbar
* fix: author should be nullable when comment was anonymous

* fix: anonymous avatars path
Co-authored-by: parabol-release-bot[bot] <150284312+parabol-release-bot[bot]@users.noreply.github.com>
* remove gcal feature flag

* remove gcal feature flag

* fix provider list
Dschoordsch and others added 27 commits January 11, 2024 12:19
* feat: Add custom category to activity library

The custom category lists all team and organization custom templates.

* Color adjustment
* chore: Update code review guidelines

* Update reviewers.yml

* Renamed "One Review Required" to "Skip Maintainer Review"

* Link to the label

* Link to correct label
* feat(kudos): send kudos by text in standups

* Remove console logs

* Fix test

* Store unicode emoji too

* Link teamPromptResponseId

* Update slack notification

* Update email notification

* Mention notification analytics

* response mentioned toast analytics

* Enable mentions in retro reflections

* feat(kudos): enable mentions in retro reflections

* feat(kudos): send kudos at the end of the retro

* Handle anonymous notifications

* Make some params optional

* make editorRef optional

* Clear undo redo stack

* remove any

* rebuild

* make name and picture nullable

* Change anonymous to someone
* feat: adding integrations to team view tabs

* fix/updating-integration-link

* Remove TeamSettingsWrapper

Only 1 route in it was still used, thus removed it

* Fix links

* Cleanup

* Don't use emotion in new components

---------

Co-authored-by: Georg Bremer <github@dschoordsch.de>
Without this fix the edit controls were shown but did not work.
* fix: highlight team dash in sidebar regardless of tab

* Forgot Organizations on mobile
* show title and type below activity card

* make activity library card dynamic

* add retro background swirls

* add background images for all meeting types

* fix custom card size

* truncate text if there is no space

* clean up

* use grape 100

* add premortem and postmortem imgs

* remove bg from categories themes

* move background img to div

* add search bar

* add getTemplateSuggestion skeleton

* show ai response to user

* handle error states

* remove ai search props

* fix vertical scroll

* fix padding

* add padding on mobile

* fix merge conflicts

* add feature flag and add json response to open ai

* make suggestedTemplate non null

* clean up

* remove activity grid from activityLibrary file

* hide default search if showing ai search

* remove unused imports
* fix: Allow multiple *NewTemplate

* Add a number to new templates

* Start counting from 1
… application work (#9335)

* chore(configuration): all non required variables commented

* PGAdmin section explained and SOCKET_PORT back and explained
Signed-off-by: Matt Krick <matt.krick@gmail.com>
* feat(kudos): show snackbar when reflection with kudos created

* Remove console log

* snackbar key
* chore: Add recurring retros feature flag

* Make it an org flag
This is helpful in the docker image for initial setup.
* add actvities to get you started title

* show desired quick start templates in order

* remove most popular

* reduce margin
Co-authored-by: parabol-release-bot[bot] <150284312+parabol-release-bot[bot]@users.noreply.github.com>
Signed-off-by: Matt Krick <matt.krick@gmail.com>
Signed-off-by: Matt Krick <matt.krick@gmail.com>
Signed-off-by: Matt Krick <matt.krick@gmail.com>
* chore: add pgvector to development environment Postgres

* fix: MailManagerDebug now always creates file paths under the O/S limit

* fix: bump rethinkdb-ts (fixes development errors on my system)

* chore: review fixes
Co-authored-by: parabol-release-bot[bot] <150284312+parabol-release-bot[bot]@users.noreply.github.com>
Signed-off-by: Matt Krick <matt.krick@gmail.com>
Co-authored-by: parabol-release-bot[bot] <150284312+parabol-release-bot[bot]@users.noreply.github.com>
Signed-off-by: Matt Krick <matt.krick@gmail.com>
Co-authored-by: parabol-release-bot[bot] <150284312+parabol-release-bot[bot]@users.noreply.github.com>
@mattkrick mattkrick merged commit c22a065 into production Jan 18, 2024
3 checks passed
@mattkrick mattkrick deleted the release/v7.13.3 branch January 18, 2024 22:50
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.