Skip to content
This repository has been archived by the owner on Sep 26, 2024. It is now read-only.

feat: Made the contributors StackedAvatars component reusable #290

Merged
merged 1 commit into from
Aug 4, 2022
Merged

feat: Made the contributors StackedAvatars component reusable #290

merged 1 commit into from
Aug 4, 2022

Conversation

OgDev-01
Copy link
Contributor

@OgDev-01 OgDev-01 commented Aug 3, 2022

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

Description

This PR replaces duplicate codes with one reusable one.

Related Tickets & Documents

closes #287
#287

Mobile & Desktop Screenshots/Recordings

Added tests?

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help

Added to documentation?

  • 📜 README.md
  • 📓 docs.opensauced.pizza
  • 🍕 dev.to/opensauced
  • 📕 storybook
  • 🙅 no documentation needed

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

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

@netlify
Copy link

netlify bot commented Aug 3, 2022 •
edited
Loading

✅ Deploy Preview for hot-sauced-ui ready!

Name Link
🔨 Latest commit c06fdb3
🔍 Latest deploy log https://app.netlify.com/sites/hot-sauced-ui/deploys/62eb0613c767ba000912cb15
😎 Deploy Preview https://deploy-preview-290--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 settings.

@0-vortex
Copy link
Contributor

0-vortex commented Aug 3, 2022

@sungoldtech (meant as a huge compliment) you are too quick! 😅 🍕

There's a pending beta -> main merge incoming, if you take a look at the beta branch, do you think it would be possible to rebase this PR against that branch/incoming change?

If not, let's get this PR on main and I will take care of the merge! 🍕

bdougie
bdougie approved these changes Aug 3, 2022
View reviewed changes
Copy link
Member

@bdougie bdougie left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for another great contribution

@0-vortex 0-vortex merged commit 05a3b15 into open-sauced:main Aug 4, 2022
github-actions bot pushed a commit that referenced this pull request Aug 4, 2022
@OgDev-01 @open-sauced
chore(minor): release 2.20.0 [skip ci]
641a625 
## [2.20.0](v2.19.0...v2.20.0) (2022-08-04)

### 🍕 Features

* make the contributors StackedAvatars component reusable ([#290](#290)) ([05a3b15](05a3b15)), closes [#287](#287)
@github-actions
Copy link

github-actions bot commented Aug 4, 2022

🎉 This PR is included in version 2.20.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@OgDev-01
Copy link
Contributor Author

OgDev-01 commented Aug 4, 2022

@sungoldtech (meant as a huge compliment) you are too quick! 😅 🍕

There's a pending beta -> main merge incoming, if you take a look at the beta branch, do you think it would be possible to rebase this PR against that branch/incoming change?

If not, let's get this PR on main and I will take care of the merge! 🍕

I'm still trying to get my hands dirty on git/github workflows.. But thanks for the compliment :)

@OgDev-01 OgDev-01 deleted the reusable-contributors-stacked-avatars-component branch August 4, 2022 00:19
@OgDev-01
Copy link
Contributor Author

OgDev-01 commented Aug 4, 2022

LGTM. Thanks for another great contribution

Pleasures are mine✌.. I'm super excited to be here😊😊

0-vortex added a commit that referenced this pull request Aug 4, 2022
@0-vortex
Merge remote-tracking branch 'origin/main' into beta
ae3a16d 
* origin/main:
  chore(minor): release 2.20.0 [skip ci]
  feat: make the contributors StackedAvatars component reusable (#290)
0-vortex added a commit that referenced this pull request Aug 11, 2022
@0-vortex
Merge remote-tracking branch 'origin/main' into alpha
e2b6177 
* origin/main: (117 commits)
  chore(patch): release 2.23.3 [skip ci]
  fix: correct swr fetcher not throwing on error (#306)
  chore(patch): release 2.23.2 [skip ci]
  fix: load more doesn't load repositories (#304)
  chore(patch): release 2.23.1 [skip ci]
  fix: dedupe app global user state (#303)
  chore(minor): release 2.23.0 [skip ci]
  feat: change title font to open sans bold (#300)
  chore(minor): release 2.22.0 [skip ci]
  feat: updated css to respect row in subnav (#298)
  feat: use async hooks and beta api for hot repos (#296)
  chore(patch): release 2.21.2 [skip ci]
  fix(vite.config.ts): correct font loading preload (#294)
  chore(patch): release 2.21.1 [skip ci]
  fix: dedupe handleVoteUpdateByRepo function (#292)
  chore(minor): release 2.21.0 [skip ci]
  chore(minor): release 2.20.0 [skip ci]
  feat: make the contributors StackedAvatars component reusable (#290)
  chore(minor): release 2.19.0 [skip ci]
  feat: added system notification (#285)
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@bdougie bdougie bdougie approved these changes

Assignees
No one assigned
Labels
released
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Feature: Make the contributors StackedAvatars component reusable
3 participants
@OgDev-01 @0-vortex @bdougie