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

feat: Create alert for paid plans #2841

Merged
merged 29 commits into from May 13, 2024
Merged

feat: Create alert for paid plans #2841

merged 29 commits into from May 13, 2024

Conversation

RulaKhaled
Copy link
Contributor

@RulaKhaled RulaKhaled commented May 6, 2024

Description

The two scenarios handled here are:

Notable Changes

Screenshots

Screenshot 2024-05-06 at 1 42 45 PM Screenshot 2024-05-06 at 1 36 11 PM

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. In 2022 this entity acquired Codecov and as result Sentry is going to need some rights from me in order to utilize my contributions in this PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

@codecov-qa
Copy link

codecov-qa bot commented May 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.48%. Comparing base (3ccb37c) to head (c2ff8d1).

✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2841   +/-   ##
=======================================
  Coverage   98.48%   98.48%           
=======================================
  Files         877      879    +2     
  Lines       12941    12951   +10     
  Branches     3469     3407   -62     
=======================================
+ Hits        12745    12755   +10     
  Misses        192      192           
  Partials        4        4           
Files Coverage Δ
...pages/RepoPage/ActivationAlert/ActivationAlert.tsx 100.00% <100.00%> (ø)
...ctivationRequiredAlert/ActivationRequiredAlert.tsx 100.00% <100.00%> (ø)
...reePlanSeatsTakenAlert/FreePlanSeatsTakenAlert.tsx 100.00% <ø> (ø)
...aidPlanSeatsTakenAlert/PaidPlanSeatsTakenAlert.tsx 100.00% <100.00%> (ø)
Components Coverage Δ
Assets 54.54% <ø> (ø)
Layouts 97.26% <ø> (ø)
Pages 99.28% <100.00%> (+<0.01%) ⬆️
Services 99.47% <ø> (ø)
Shared 99.84% <ø> (ø)
UI 94.53% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3ccb37c...c2ff8d1. Read the comment docs.

Copy link

codecov bot commented May 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.49%. Comparing base (3ccb37c) to head (c2ff8d1).

✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #2841   +/-   ##
=====================================
  Coverage   98.49   98.49           
=====================================
  Files        877     879    +2     
  Lines      12941   12951   +10     
  Branches    3403    3408    +5     
=====================================
+ Hits       12745   12755   +10     
  Misses       192     192           
  Partials       4       4           
Files Coverage Δ
...pages/RepoPage/ActivationAlert/ActivationAlert.tsx 100.00% <100.00%> (ø)
...ctivationRequiredAlert/ActivationRequiredAlert.tsx 100.00% <100.00%> (ø)
...reePlanSeatsTakenAlert/FreePlanSeatsTakenAlert.tsx 100.00% <ø> (ø)
...aidPlanSeatsTakenAlert/PaidPlanSeatsTakenAlert.tsx 100.00% <100.00%> (ø)
Components Coverage Δ
Assets 54.54% <ø> (ø)
Layouts 97.26% <ø> (ø)
Pages 99.28% <100.00%> (+<0.01%) ⬆️
Services 99.47% <ø> (ø)
Shared 99.84% <ø> (ø)
UI 94.53% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3ccb37c...c2ff8d1. Read the comment docs.

Copy link

codecov-public-qa bot commented May 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.48%. Comparing base (3ccb37c) to head (c2ff8d1).

✅ All tests successful. No failed tests found ☺️

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2841   +/-   ##
=======================================
  Coverage   98.48%   98.48%           
=======================================
  Files         877      879    +2     
  Lines       12941    12951   +10     
  Branches     3464     3468    +4     
=======================================
+ Hits        12745    12755   +10     
  Misses        192      192           
  Partials        4        4           
Files Coverage Δ
...pages/RepoPage/ActivationAlert/ActivationAlert.tsx 100.00% <100.00%> (ø)
...ctivationRequiredAlert/ActivationRequiredAlert.tsx 100.00% <100.00%> (ø)
...reePlanSeatsTakenAlert/FreePlanSeatsTakenAlert.tsx 100.00% <ø> (ø)
...aidPlanSeatsTakenAlert/PaidPlanSeatsTakenAlert.tsx 100.00% <100.00%> (ø)
Components Coverage Δ
Assets 54.54% <ø> (ø)
Layouts 97.26% <ø> (ø)
Pages 99.28% <100.00%> (+<0.01%) ⬆️
Services 99.47% <ø> (ø)
Shared 99.84% <ø> (ø)
UI 94.53% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3ccb37c...c2ff8d1. Read the comment docs.

Copy link

codecov bot commented May 6, 2024

Bundle Report

Changes will increase total bundle size by 1.67kB ⬆️

Bundle name Size Change
gazebo-production-array-push 6.61MB 1.67kB ⬆️

@codecov-staging
Copy link

codecov-staging bot commented May 6, 2024

Bundle Report

Changes will increase total bundle size by 1.67kB ⬆️

Bundle name Size Change
gazebo-staging-array-push 6.61MB 1.67kB ⬆️

@codecov-releaser
Copy link
Contributor

codecov-releaser commented May 6, 2024

✅ Deploy preview for gazebo ready!

Previews expire after 1 month automatically.

Commit Created Cloud Enterprise
ec14c36 Mon, 06 May 2024 11:59:42 GMT Expired Expired
2707208 Mon, 13 May 2024 14:22:25 GMT Expired Expired
2707208 Mon, 13 May 2024 14:23:09 GMT Expired Expired
c2ff8d1 Mon, 13 May 2024 14:51:12 GMT Cloud Enterprise

nicholas-codecov and others added 22 commits May 13, 2024 16:06
* update usePrefetchBranchDirEntry

* update useRepoBranchContents

* update index file

* update query off of repositoryDeprecated

* update tests

* small change to have commit signed

* update constants to ts

* small tweaks to the query keys
* feat: Add activation banner for trial eligible owners

* pull out interface + spec stuff

* Update to reflect paid plan activation banner

* Refactor CircleCI repo onboarding into one file (#2806)

* Refactor Other CI repo onboarding into one file (#2807)

* Update repo onboarding title position and page alignment (#2818)

* sec: 390 - Add validation for potential XSS vuln (#2797)

* add tests, and validation for provider

* add back supportServiceless param

* ref: 1548 Part 1: Convert all Header files to TS (#2821)

* ref all header files to TS

* remove prop types and rebase

* fix: Remove repository from GUT settings page header (#2823)

Small tweak removing `repository` from the GUT settings page.

* Install radix-ui react radio group (#2825)

* Update repo onboarding steps with new Card component (#2819)

GH codecov/engineering-team#1665

* feat: Add hasSeatsLeft to plan query

* Update to reflect SeatsLimitReachedBanner

* feat: paid plan activation banner

* update with from FreePlanSeatsLimitBanner

* value duplicate

* feat: paid plan seats limit banner

* just one more small tweak

* update name to FreePlanSeatsLimitBanner

* remove queryclient call

* update tests

* fix padding

* Update to have a const for plan value

---------

Co-authored-by: Spencer Murray <159931558+spalmurray-codecov@users.noreply.github.com>
Co-authored-by: ajay-sentry <159853603+ajay-sentry@users.noreply.github.com>
Co-authored-by: nicholas-codecov <nicholas.deschenes@sentry.io>
* feat: Add activation banner for trial eligible owners

* pull out interface + spec stuff

* Update to reflect paid plan activation banner

* Refactor CircleCI repo onboarding into one file (#2806)

* Refactor Other CI repo onboarding into one file (#2807)

* Update repo onboarding title position and page alignment (#2818)

* sec: 390 - Add validation for potential XSS vuln (#2797)

* add tests, and validation for provider

* add back supportServiceless param

* ref: 1548 Part 1: Convert all Header files to TS (#2821)

* ref all header files to TS

* remove prop types and rebase

* fix: Remove repository from GUT settings page header (#2823)

Small tweak removing `repository` from the GUT settings page.

* Install radix-ui react radio group (#2825)

* Update repo onboarding steps with new Card component (#2819)

GH codecov/engineering-team#1665

* feat: Add hasSeatsLeft to plan query

* Update to reflect SeatsLimitReachedBanner

* feat: paid plan activation banner

* update with from FreePlanSeatsLimitBanner

* value duplicate

* feat: Activation required banner

* clean up previous commit

* match design

* fix style

* Resolve conflicts

---------

Co-authored-by: Spencer Murray <159931558+spalmurray-codecov@users.noreply.github.com>
Co-authored-by: ajay-sentry <159853603+ajay-sentry@users.noreply.github.com>
Co-authored-by: nicholas-codecov <nicholas.deschenes@sentry.io>
…2844)

* Convert UseCommitYaml to Repository from RepositoryDeprecated

* Update query key
…2847)

* init conversion and remove deprecated, testing

* fix tests for useCommitErrors

* add additional spec

* add dev stuff

* update test coverage
* add plan route logic if to param set as plan

* help w/ test

* remove some unneeded mocks

* remove to

* fix spec mocks
* Update usereposettings

* Update tests

* update test

* Update more tests

* Update more tests

* more test updates

* Update type

* Update

* Update query

* Refactor
* pin @sentry/react to the latest release candidated

* update lockfile
* Fix coverage tab file list table

* Fix coverage tab code tree table tests

* Fix pull indirect changed files table

* Fix commit file explorer table

* Fix missed instance of getByText
* Convert Access.jsx -> tsx

* Fix type issue with TokensTable

* Remove unnecessary import

* Convert SessionsTable to ts

* Add tests

* Fix tests

* Fix all testLocation: any

* Use msw for mutation mock

* Improve flatmap callback for readability

* Better type hint for flat map return

* Fix flake

* Fix imports

* Fix indirect coverage loss
* Fix copy pasta border/padding

* Page width change

* Vite onboarding overhaul

* Webpack onboarding overhaul

* Rollup onboarding overhaul

* Fix tests
…ds (#2865)

* Update to 8 uploads before comment to avoid many comments

* Oopsie it's actually the ceiling
…oryDeprecated (#2857)

* fix tests, confirm it works

* export type and use instead
* Use RadioTileGroup for navigation on the repo onboarding page

* Pull getInitalProvider out of component

* Fix tests

* Remove unnecessary div
* Clean up pull file explorer tests

* Fix tests

* Revert "Fix tests"

This reverts commit f5aeccd.
#2869)

* Fix responsiveness issue on RadioTileGroup

* Add longer description to story

* Left align the label text
nicholas-codecov and others added 3 commits May 13, 2024 16:06
* update usePrefetchSingleFileComp to TS

* update NameColumn tests

* move fragment schema next to gql fragment string
@codecov-notifications
Copy link

codecov-notifications bot commented May 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2841   +/-   ##
=======================================
  Coverage   98.48%   98.48%           
=======================================
  Files         877      879    +2     
  Lines       12941    12951   +10     
  Branches     3404     3468   +64     
=======================================
+ Hits        12745    12755   +10     
  Misses        192      192           
  Partials        4        4           
Files Coverage Δ
...pages/RepoPage/ActivationAlert/ActivationAlert.tsx 100.00% <100.00%> (ø)
...ctivationRequiredAlert/ActivationRequiredAlert.tsx 100.00% <100.00%> (ø)
...reePlanSeatsTakenAlert/FreePlanSeatsTakenAlert.tsx 100.00% <ø> (ø)
...aidPlanSeatsTakenAlert/PaidPlanSeatsTakenAlert.tsx 100.00% <100.00%> (ø)
Components Coverage Δ
Assets 54.54% <ø> (ø)
Layouts 97.26% <ø> (ø)
Pages 99.28% <100.00%> (+<0.01%) ⬆️
Services 99.47% <ø> (ø)
Shared 99.84% <ø> (ø)
UI 94.53% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3ccb37c...c2ff8d1. Read the comment docs.

@@ -3,6 +3,9 @@ import { useParams } from 'react-router-dom'
import { usePlanData } from 'services/account'
import { isFreePlan } from 'shared/utils/billing'

import ActivationRequiredAlert from './ActivationRequiredAlert'
import FreePlanSeatsTakenAlert from './FreePlanSeatsTakenAlert'
import PaidPlanSeatsTakenAlert from './PaidPlanSeatsTakenAlert'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

might be a leftover from a merge

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yah my bad 🏃‍♀️

@RulaKhaled RulaKhaled added this pull request to the merge queue May 13, 2024
Merged via the queue into main with commit 4702b47 May 13, 2024
60 checks passed
@RulaKhaled RulaKhaled deleted the alert-for-paid-plans branch May 13, 2024 15:00
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.

None yet

7 participants