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

fix: now OSCR column is hidden if feature flag is not enabled for OSCR #3676

Merged
merged 4 commits into from
Jul 3, 2024

Conversation

nickytonline
Copy link
Member

@nickytonline nickytonline commented Jul 3, 2024

Description

OSCR data is hidden behind a feature flag, but the column wasn't. This PR removes the Rating column if the feature flag is not enabled.

Related Tickets & Documents

Fixes #3677

Mobile & Desktop Screenshots/Recordings

Before

Contributor Insight (Workspaces)

CleanShot 2024-07-03 at 11 07 15

Explore page (contributors)

CleanShot 2024-07-03 at 11 02 08

After

Contributor Insight (Workspaces)

CleanShot 2024-07-03 at 11 08 35

Explore page (contributors)

CleanShot 2024-07-03 at 11 01 26

Steps to QA

  1. Go to the Explore page contributors tab.
  2. Notice there is no longer a Rating column.
  3. Logged in, go to one of your contributor insights
  4. Notice there is a Rating column.
  5. Log out and refresh the page.
  6. Notice there is no longer a Rating column.

Tier (staff will fill in)

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

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

Copy link

netlify bot commented Jul 3, 2024

Deploy Preview for design-insights ready!

Name Link
🔨 Latest commit 6e6c1ee
🔍 Latest deploy log https://app.netlify.com/sites/design-insights/deploys/66856bca33730e00083d1df6
😎 Deploy Preview https://deploy-preview-3676--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 Jul 3, 2024

Deploy Preview for oss-insights ready!

Name Link
🔨 Latest commit 6e6c1ee
🔍 Latest deploy log https://app.netlify.com/sites/oss-insights/deploys/66856bca1c964500081c1b44
😎 Deploy Preview https://deploy-preview-3676--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.

@nickytonline nickytonline requested review from isabensusan, BekahHW and a team July 3, 2024 15:06
@bdougie
Copy link
Member

bdougie commented Jul 3, 2024

Can you confirm if you still see the table alignment issue?

production
Screenshot 2024-07-03 at 8 03 36 AM

beta
Screenshot 2024-07-03 at 8 06 04 AM

update: I see it in your OP screenshot. I expect that it should be aligned.

@nickytonline nickytonline merged commit 061c590 into beta Jul 3, 2024
13 checks passed
@nickytonline nickytonline deleted the nickytonline/hide-oscr-colum branch July 3, 2024 17:21
open-sauced bot pushed a commit that referenced this pull request Jul 3, 2024
## [2.41.1-beta.1](v2.41.0...v2.41.1-beta.1) (2024-07-03)

### 🐛 Bug Fixes

* now OSCR column is hidden if feature flag is not enabled for OSCR ([#3676](#3676)) ([061c590](061c590))
open-sauced bot pushed a commit that referenced this pull request Jul 4, 2024
## [2.42.0](v2.41.0...v2.42.0) (2024-07-04)

### 🍕 Features

* add repo filtering to contributions tables on contributor profile ([#3580](#3580)) ([a2e49ef](a2e49ef))

### 🐛 Bug Fixes

* lottery factor copy ([#3682](#3682)) ([baac008](baac008))
* now OSCR column is hidden if feature flag is not enabled for OSCR ([#3676](#3676)) ([061c590](061c590))
* now the contributor insights new/edit page fits in smaller screens ([#3679](#3679)) ([256af40](256af40))
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.

Bug: OSCR column visible in contributor list when feature flag is disabled
3 participants