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

refactor: show banner across workspace when limits are exceeded #3233

Merged
merged 4 commits into from Apr 25, 2024

Conversation

brandonroberts
Copy link
Contributor

@brandonroberts brandonroberts commented Apr 23, 2024

Description

  • Displays the workspace upgrade banner across the entire workspace if not upgraded and overall thresholds have been exceeded
  • Uses the API instead of client-side code for displaying the banner. Needs https://github.com/open-sauced/api/pull/739
  • Updated the copy to be workspace-wide instead of insight-specific

Related Tickets & Documents

Closes #3210

Mobile & Desktop Screenshots/Recordings

image image image

CleanShot 2024-04-25 at 10 42 29

Steps to QA

  1. Login
  2. Go to a workspace you own that has more than 100 repositories in an insight page or 10 contributors in an insight page
  3. Note the upgrade banner is always displayed

Tier (staff will fill in)

  • Tier 1
  • Tier 2
  • Tier 3
  • Tier 4

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

@brandonroberts brandonroberts requested a review from a team April 23, 2024 20:05
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

PR Compliance Checks

Thank you for your Pull Request! We have run several checks on this pull request in order to make sure it's suitable for merging into this project. The results are listed in the following section.

Issue Reference

In order to be considered for merging, the pull request description must refer to a specific issue number. This is described in our Contributing Guide.
This check is looking for a phrase similar to: "Fixes #XYZ" or "Resolves #XYZ" where XYZ is the issue number that this PR is meant to address.

Copy link

netlify bot commented Apr 23, 2024

Deploy Preview for design-insights ready!

Name Link
🔨 Latest commit 7c80159
🔍 Latest deploy log https://app.netlify.com/sites/design-insights/deploys/662a6cb72ce9080007d67149
😎 Deploy Preview https://deploy-preview-3233--design-insights.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Apr 23, 2024

Deploy Preview for oss-insights ready!

Name Link
🔨 Latest commit 7c80159
🔍 Latest deploy log https://app.netlify.com/sites/oss-insights/deploys/662a6cb769ce0600076270f2
😎 Deploy Preview https://deploy-preview-3233--oss-insights.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Member

@nickytonline nickytonline left a comment

Choose a reason for hiding this comment

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

This looks good. The only thing we need to handle is on smaller screens, the modal will be too large. We can use the drawer component for that. I can sync with you in the AM about that @brandonroberts.

@bdougie
Copy link
Member

bdougie commented Apr 24, 2024

image

What I see in mobile. Iphone 15

  • workspace name is covered by banner

@nickytonline
Copy link
Member

The banner should be good now @bdougie @brandonroberts

CleanShot 2024-04-25 at 10 42 29

@brandonroberts brandonroberts merged commit f0d1dac into beta Apr 25, 2024
19 checks passed
@brandonroberts brandonroberts deleted the refactor-workspace-upgrade branch April 25, 2024 14:55
open-sauced bot pushed a commit that referenced this pull request Apr 25, 2024
## [2.21.0-beta.6](v2.21.0-beta.5...v2.21.0-beta.6) (2024-04-25)

### 🧑‍💻 Code Refactoring

* show banner across workspace when limits are exceeded ([#3233](#3233)) ([f0d1dac](f0d1dac))
open-sauced bot pushed a commit that referenced this pull request Apr 25, 2024
## [2.21.0](v2.20.0...v2.21.0) (2024-04-25)

### 🐛 Bug Fixes

* now StarSearch responses with links have styled links ([#3236](#3236)) ([99e622e](99e622e))
* now suggestion prompts run immediately ([#3242](#3242)) ([662d0b5](662d0b5))

### 🍕 Features

* add OG image to StarSearch waitlist ([#3245](#3245)) ([8563264](8563264))
* add share button to repo pages ([#3235](#3235)) ([cde9d62](cde9d62))
* implemented StarSearch waitlist ([#3231](#3231)) ([654d525](654d525))
* update stars and forks stats on repo pages ([#3199](#3199)) ([43ffce4](43ffce4))

### 🧑‍💻 Code Refactoring

* show banner across workspace when limits are exceeded ([#3233](#3233)) ([f0d1dac](f0d1dac))
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.

Feature: Always show workspace upgrade banner when over limit
4 participants