Skip to content

feat: Remove plan feedback banners#4005

Open
calvin-codecov wants to merge 2 commits into
mainfrom
cy/remove_survey_banners
Open

feat: Remove plan feedback banners#4005
calvin-codecov wants to merge 2 commits into
mainfrom
cy/remove_survey_banners

Conversation

@calvin-codecov

Copy link
Copy Markdown
Contributor

Remove survey banners relating to Pro and Team plans

@codecov

codecov Bot commented Jun 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.73%. Comparing base (7a6fac1) to head (74244a1).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4005      +/-   ##
==========================================
- Coverage   98.74%   98.73%   -0.01%     
==========================================
  Files         830      828       -2     
  Lines       15174    15158      -16     
  Branches     4364     4362       -2     
==========================================
- Hits        14983    14967      -16     
  Misses        184      184              
  Partials        7        7              
Files with missing lines Coverage Δ
src/shared/GlobalTopBanners/GlobalTopBanners.tsx 100.00% <ø> (ø)
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.37% <ø> (ø)
Services 99.33% <ø> (ø)
Shared 98.94% <ø> (-0.02%) ⬇️
UI 99.01% <ø> (ø)

Continue to review full report in Codecov by Harness.

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

@codecov-public-qa

Copy link
Copy Markdown

❌ 3 Tests Failed:

Tests completed Failed Passed Skipped
2394 3 2391 2
View the top 3 failed test(s) by shortest run time
src/shared/GlobalTopBanners/GlobalTopBanners.test.tsx > GlobalTopBanners > renders pro plan feedback banner
Stack Traces | 1s run time
TestingLibraryElementError: Unable to find an element with the text: /ProPlanFeedbackBanner/. This could be because the text is broken up by multiple elements. In this case, you can provide a function for your text matcher to make your matcher more flexible.

Ignored nodes: comments, script, style
<body>
  <div>
    <div
      class="[&>*:last-child]:block"
    >
      AnnouncementBanner
      OktaBanners
      TrialBanner
      BundleFeedbackBanner
      TokenlessBanner
    </div>
  </div>
</body>

Ignored nodes: comments, script, style
<body>
  <div>
    <div
      class="[&>*:last-child]:block"
    >
      AnnouncementBanner
      OktaBanners
      TrialBanner
      BundleFeedbackBanner
      TokenlessBanner
    </div>
  </div>
</body>waitForWrapper node_modules/@.../dom/dist/wait-for.js:163:27node_modules/@.../dom/dist/query-helpers.js:86:33
 ❯ .../shared/GlobalTopBanners/GlobalTopBanners.test.tsx:45:33
src/shared/GlobalTopBanners/GlobalTopBanners.test.tsx > GlobalTopBanners > renders bundle feedback banner
Stack Traces | 1s run time
TestingLibraryElementError: Unable to find an element with the text: /ProPlanFeedbackBanner/. This could be because the text is broken up by multiple elements. In this case, you can provide a function for your text matcher to make your matcher more flexible.

Ignored nodes: comments, script, style
<body>
  <div>
    <div
      class="[&>*:last-child]:block"
    >
      AnnouncementBanner
      OktaBanners
      TrialBanner
      BundleFeedbackBanner
      TokenlessBanner
    </div>
  </div>
</body>

Ignored nodes: comments, script, style
<body>
  <div>
    <div
      class="[&>*:last-child]:block"
    >
      AnnouncementBanner
      OktaBanners
      TrialBanner
      BundleFeedbackBanner
      TokenlessBanner
    </div>
  </div>
</body>waitForWrapper node_modules/@.../dom/dist/wait-for.js:163:27node_modules/@.../dom/dist/query-helpers.js:86:33
 ❯ .../shared/GlobalTopBanners/GlobalTopBanners.test.tsx:52:33
src/shared/GlobalTopBanners/GlobalTopBanners.test.tsx > GlobalTopBanners > renders team plan feedback banner
Stack Traces | 1.01s run time
TestingLibraryElementError: Unable to find an element with the text: /TeamPlanFeedbackBanner/. This could be because the text is broken up by multiple elements. In this case, you can provide a function for your text matcher to make your matcher more flexible.

Ignored nodes: comments, script, style
<body>
  <div>
    <div
      class="[&>*:last-child]:block"
    >
      AnnouncementBanner
      OktaBanners
      TrialBanner
      BundleFeedbackBanner
      TokenlessBanner
    </div>
  </div>
</body>

Ignored nodes: comments, script, style
<body>
  <div>
    <div
      class="[&>*:last-child]:block"
    >
      AnnouncementBanner
      OktaBanners
      TrialBanner
      BundleFeedbackBanner
      TokenlessBanner
    </div>
  </div>
</body>waitForWrapper node_modules/@.../dom/dist/wait-for.js:163:27node_modules/@.../dom/dist/query-helpers.js:86:33
 ❯ .../shared/GlobalTopBanners/GlobalTopBanners.test.tsx:38:33

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@codecov

codecov Bot commented Jun 18, 2026

Copy link
Copy Markdown

Bundle Report

Changes will decrease total bundle size by 1.38kB (-0.01%) ⬇️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
gazebo-production-esm 5.96MB -767 bytes (-0.01%) ⬇️
gazebo-production-system 5.88MB -610 bytes (-0.01%) ⬇️

Affected Assets, Files, and Routes:

view changes for bundle: gazebo-production-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index.*.js 49 bytes 32.31kB 0.15%
assets/index.*.js 53 bytes 126.59kB 0.04%
assets/index.*.js 53 bytes 90.08kB 0.06%
assets/index.*.js -1.98kB 639.02kB -0.31%
assets/index.*.js 53 bytes 22.22kB 0.24%
assets/index.*.js 53 bytes 41.73kB 0.13%
assets/index.*.js 53 bytes 36.81kB 0.14%
assets/index.*.js 49 bytes 31.33kB 0.16%
assets/index.*.js 53 bytes 125.73kB 0.04%
assets/BillingDetails.*.js -5 bytes 29.71kB -0.02%
assets/ReposTable.*.js 53 bytes 11.3kB 0.47%
assets/useIsTeamPlan.*.js (New) 751 bytes 751 bytes 100.0% 🚀

Files in assets/index.*.js:

  • ./src/shared/GlobalTopBanners/GlobalTopBanners.tsx → Total Size: 955 bytes
view changes for bundle: gazebo-production-system

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-legacy.*.js 55 bytes 31.69kB 0.17%
assets/index-legacy.*.js -1.8kB 716.68kB -0.25%
assets/index-legacy.*.js 57 bytes 84.82kB 0.07%
assets/index-legacy.*.js 54 bytes 35.27kB 0.15%
assets/index-legacy.*.js 61 bytes 113.13kB 0.05%
assets/index-legacy.*.js 2 bytes 115.09kB 0.0%
assets/index-legacy.*.js 54 bytes 41.23kB 0.13%
assets/index-legacy.*.js 54 bytes 20.94kB 0.26%
assets/index-legacy.*.js 55 bytes 29.58kB 0.19%
assets/ReposTable-legacy.*.js 54 bytes 10.54kB 0.51%
assets/useIsTeamPlan-legacy.*.js (New) 746 bytes 746 bytes 100.0% 🚀

Files in assets/index-legacy.*.js:

  • ./src/shared/GlobalTopBanners/GlobalTopBanners.tsx → Total Size: 955 bytes

@codecov-releaser

codecov-releaser commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

✅ Deploy preview for gazebo ready!

Previews expire after 1 month automatically.

Storybook

Commit Created Cloud Enterprise
74244a1 Thu, 18 Jun 2026 23:07:30 GMT Expired Expired
74244a1 Thu, 18 Jun 2026 23:12:22 GMT Cloud Enterprise

@codecov-notifications

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

@@            Coverage Diff             @@
##             main    #4005      +/-   ##
==========================================
- Coverage   98.74%   98.73%   -0.01%     
==========================================
  Files         830      828       -2     
  Lines       15174    15158      -16     
  Branches     4364     4362       -2     
==========================================
- Hits        14983    14967      -16     
  Misses        184      184              
  Partials        7        7              
Files with missing lines Coverage Δ
src/shared/GlobalTopBanners/GlobalTopBanners.tsx 100.00% <ø> (ø)
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.37% <ø> (ø)
Services 99.33% <ø> (ø)
Shared 98.94% <ø> (-0.02%) ⬇️
UI 99.01% <ø> (ø)

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 7a6fac1...74244a1. Read the comment docs.

@codecov-qa

codecov-qa Bot commented Jun 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.73%. Comparing base (7a6fac1) to head (74244a1).

@@            Coverage Diff             @@
##             main    #4005      +/-   ##
==========================================
- Coverage   98.74%   98.73%   -0.01%     
==========================================
  Files         830      828       -2     
  Lines       15174    15158      -16     
  Branches     4372     4362      -10     
==========================================
- Hits        14983    14967      -16     
  Misses        184      184              
  Partials        7        7              
Files with missing lines Coverage Δ
src/shared/GlobalTopBanners/GlobalTopBanners.tsx 100.00% <ø> (ø)
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.37% <ø> (ø)
Services 99.33% <ø> (ø)
Shared 98.94% <ø> (-0.02%) ⬇️
UI 99.01% <ø> (ø)

Continue to review full report in Codecov by Harness.

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

3 participants