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: Removing the votes feature #518

Merged
merged 6 commits into from
Apr 1, 2024
Merged

feat: Removing the votes feature #518

merged 6 commits into from
Apr 1, 2024

Conversation

bdougie
Copy link
Member

@bdougie bdougie commented Apr 1, 2024

Description

This feature has not really been leveraged and the OpenSauced project has moved towards providing insights rather than getting signal from user usage. This PR removes all references and functionality for repo voting.

What type of PR is this? (check all applicable)

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation Update
  • 🎨 Style
  • πŸ§‘β€πŸ’» Code Refactor
  • πŸ”₯ Performance Improvements
  • βœ… Test
  • πŸ€– Build
  • πŸ” CI
  • πŸ“¦ Chore (Release)
  • ⏩ Revert

Related Tickets & Documents

Closes #517

Mobile & Desktop Screenshots/Recordings

before:
Screen Shot 2024-04-01 at 2 28 53 PM

after:

Screen Shot 2024-04-01 at 2 28 39 PM

Steps to QA

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?

giphy (20)

Copy link

netlify bot commented Apr 1, 2024

βœ… Deploy Preview for hot-sauced-ui ready!

Name Link
πŸ”¨ Latest commit 74494fa
πŸ” Latest deploy log https://app.netlify.com/sites/hot-sauced-ui/deploys/660b2d7cfe70a20008b81604
😎 Deploy Preview https://deploy-preview-518--hot-sauced-ui.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.

@bdougie
Copy link
Member Author

bdougie commented Apr 1, 2024

Just a FYI @jpmcb. We will no longer need any Votes endpoints after this PR. Nothing actionable today, but going to move forward to clean this up while this is still pointing traffic to our app.

Next up is connecting the endpoint #476

We will still want to move this to a new explore page experience, but until then I'd rather have this tested somewhere.

@bdougie bdougie merged commit 8db6b23 into beta Apr 1, 2024
11 checks passed
@bdougie bdougie deleted the votes-removed branch April 1, 2024 22:07
github-actions bot pushed a commit that referenced this pull request Apr 1, 2024
## [2.36.0-beta.3](v2.36.0-beta.2...v2.36.0-beta.3) (2024-04-01)

### πŸ• Features

* Removing the votes feature ([#518](#518)) ([8db6b23](8db6b23))
github-actions bot pushed a commit that referenced this pull request Apr 1, 2024
## [2.36.0](v2.35.0...v2.36.0) (2024-04-01)

### πŸ• Features

* Removing the votes feature ([#518](#518)) ([8db6b23](8db6b23))
* Replaced print auth token with copy to clipboard ([#514](#514)) ([7684b81](7684b81))
* update links to use app ([#516](#516)) ([cb9eea0](cb9eea0))
Copy link

github-actions bot commented Apr 1, 2024

πŸŽ‰ This PR is included in version 2.36.0 πŸŽ‰

The release is available on:

Your semantic-release bot πŸ“¦πŸš€

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: Removing votes
1 participant