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

feat: Add new chapter on teams #27

Merged
merged 46 commits into from
Jan 19, 2024
Merged

feat: Add new chapter on teams #27

merged 46 commits into from
Jan 19, 2024

Conversation

BekahHW
Copy link
Member

@BekahHW BekahHW commented Jan 7, 2024

Description

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 #14

Mobile & Desktop Screenshots/Recordings

Steps to QA

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?

@BekahHW BekahHW marked this pull request as ready for review January 10, 2024 16:19
@BekahHW BekahHW requested review from jdwilkin4 and adiati98 January 10, 2024 16:19
your-team.md Outdated Show resolved Hide resolved
Copy link
Contributor

@jdwilkin4 jdwilkin4 left a comment

Choose a reason for hiding this comment

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

I think this looks great πŸ’―

@jdwilkin4 jdwilkin4 requested a review from adiati98 January 10, 2024 21:00
Copy link
Member

@adiati98 adiati98 left a comment

Choose a reason for hiding this comment

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

@BekahHW sorry for the long reviews! But in some of them, I'm asking for your thoughts as well. 😊

This looks awesome and I love what you wrote here! I learn so many new things through this section and they're very helpful! πŸ˜ƒ

your-team.md Outdated Show resolved Hide resolved
your-team.md Outdated Show resolved Hide resolved
your-team.md Outdated Show resolved Hide resolved
your-team.md Outdated Show resolved Hide resolved
your-team.md Outdated Show resolved Hide resolved
your-team.md Show resolved Hide resolved
your-team.md Outdated Show resolved Hide resolved
your-team.md Outdated Show resolved Hide resolved
your-team.md Outdated Show resolved Hide resolved
your-team.md Outdated Show resolved Hide resolved
@BekahHW BekahHW requested a review from adiati98 January 19, 2024 14:52
@BekahHW
Copy link
Member Author

BekahHW commented Jan 19, 2024

@adiati98 I think I've addressed all the comments. I really wish GH had a unified view for all the comments. It would actually be great to see them in VSCode so I know I'm getting everything.

@adiati98
Copy link
Member

@adiati98 I think I've addressed all the comments. I really wish GH had a unified view for all the comments. It would actually be great to see them in VSCode so I know I'm getting everything.

Yeah, that would be awesome. πŸ˜„

Copy link
Member

@adiati98 adiati98 left a comment

Choose a reason for hiding this comment

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

All are fixed. LGTM. Thank you, @BekahHW! πŸš€

@adiati98 adiati98 merged commit b443f5c into main Jan 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: Your Team
3 participants