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: Change announcement card on highlight feed to starsearch #3313

Merged
merged 4 commits into from May 6, 2024

Conversation

BekahHW
Copy link
Member

@BekahHW BekahHW commented May 3, 2024

Description

  • Changes the announcement card on the feed page to the waitlist for starSearch

Related Tickets & Documents

closes #3312

Mobile & Desktop Screenshots/Recordings

image

Steps to QA

If you have it running locally go to http://localhost:3000/feed

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?

Copy link

netlify bot commented May 3, 2024

Deploy Preview for oss-insights ready!

Name Link
🔨 Latest commit b424de6
🔍 Latest deploy log https://app.netlify.com/sites/oss-insights/deploys/6635b1e96f7cd3000845c45b
😎 Deploy Preview https://deploy-preview-3313--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

netlify bot commented May 3, 2024

Deploy Preview for design-insights ready!

Name Link
🔨 Latest commit b424de6
🔍 Latest deploy log https://app.netlify.com/sites/design-insights/deploys/6635b1e99a365e0008b36496
😎 Deploy Preview https://deploy-preview-3313--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.

@brandonroberts
Copy link
Contributor

This E2E test needs updating for the title of that section.

https://github.com/open-sauced/app/blob/beta/e2e/homepage.spec.ts#L16

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.

I'll update the test.

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.

The image looks grainy, probably because it's the actual OG image. Can we get a better one for the size here @isabensusan?

CleanShot 2024-05-03 at 11 39 48

@isabensusan
Copy link
Member

isabensusan commented May 3, 2024

banner

@nickytonline
Copy link
Member

Now the announcment card is less grainy.

CleanShot 2024-05-03 at 23 55 08

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.

:shipit:

@isabensusan isabensusan self-requested a review May 6, 2024 15:21
@nickytonline nickytonline merged commit a38af76 into beta May 6, 2024
13 checks passed
@nickytonline nickytonline deleted the chore/update-card-starsearch branch May 6, 2024 15:28
open-sauced bot pushed a commit that referenced this pull request May 6, 2024
## [2.24.2-beta.1](v2.24.1...v2.24.2-beta.1) (2024-05-06)

### 🐛 Bug Fixes

* Change announcement card on highlight feed to starsearch ([#3313](#3313)) ([a38af76](a38af76))
nickytonline added a commit that referenced this pull request May 7, 2024
nickytonline pushed a commit that referenced this pull request May 7, 2024
* Change announcement card on highlight feed to starsearch ([#3313](#3313)) ([a38af76](a38af76))
open-sauced bot pushed a commit that referenced this pull request May 7, 2024
## [2.25.0](v2.24.2...v2.25.0) (2024-05-07)

### 🐛 Bug Fixes

* Change announcement card on highlight feed to starsearch ([#3313](#3313)) ([a38af76](a38af76))
* Star-search md rendering renders spacing from LLM ([#3326](#3326)) ([7bf27f1](7bf27f1))
* style and layout fixes for StarSearch ([#3304](#3304)) ([599d8f6](599d8f6))

### 🍕 Features

* Issues and PR charts for Repo pages ([#3323](#3323)) ([97a6239](97a6239))
* search and select for workspaces in sidebar ([#3307](#3307)) ([5d91aaf](5d91aaf))
* uniform styling for Repo page charts ([#3306](#3306)) ([9cd008e](9cd008e))
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.

Change ad card on highlights feed to starSearch waitlist
4 participants