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: Highlight Cards UI Generation (frontend-only) #36

Merged
merged 12 commits into from
May 2, 2023

Conversation

Deadreyo
Copy link
Contributor

@Deadreyo Deadreyo commented Apr 26, 2023

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

  • Heavily refractors the file structure of the project
  • Adds highlight cards UI services and testing

Related Tickets & Documents

closes #35

Mobile & Desktop Screenshots/Recordings

101
102
103

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?

@Deadreyo Deadreyo marked this pull request as ready for review April 29, 2023 18:50
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

PR Compliance Checks

Thank you for your Pull Request! We have run several checks on this pull request in order to make sure it's suitable for merging into this project. The results are listed in the following section.

Watched Files

This pull request modifies specific files that require careful review by the maintainers.

Files Matched

  • package.json

@Deadreyo Deadreyo requested a review from 0-vortex April 29, 2023 18:53
Copy link
Contributor

@0-vortex 0-vortex left a comment

Choose a reason for hiding this comment

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

Very good work decoupling the cards, we should outsource the icons to the same design system as insights if we envision more than the heart emoji used and potentially experimenting with the alpha api for extended user information when doing the backend.

It can go to beta with just a small lint clean-up and improvement issues since this is just tackling the frontend side of things! πŸ‘

@Deadreyo Deadreyo requested a review from 0-vortex April 30, 2023 17:04
@Deadreyo
Copy link
Contributor Author

@0-vortex fixed the lint issues & merge conflicts

Copy link
Contributor

@0-vortex 0-vortex left a comment

Choose a reason for hiding this comment

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

LGTM! πŸ‘

@bdougie
Copy link
Member

bdougie commented May 2, 2023

It is not clear what "frontend-only" means in the title. Can you clarify that?

Other than that, functionality of the card works. Also is there a test plan or steps to review or test the cards?

@Deadreyo
Copy link
Contributor Author

Deadreyo commented May 2, 2023

It is not clear what "frontend-only" means in the title. Can you clarify that?

Meaning that this PR is only responsible for only generating the cards, and not responsible for connecting to the backend or database.

Other than that, functionality of the card works. Also is there a test plan or steps to review or test the cards?

There are scripts to manually generate and test the social cards visually , test:local:user and test:local:highlight

@Deadreyo Deadreyo merged commit 138a847 into beta May 2, 2023
5 checks passed
@Deadreyo Deadreyo deleted the 35-feature-highlight-cards branch May 2, 2023 06:14
github-actions bot pushed a commit that referenced this pull request May 2, 2023
## [2.1.0-beta.3](v2.1.0-beta.2...v2.1.0-beta.3) (2023-05-02)

### πŸ• Features

* Highlight Cards UI Generation (frontend-only) ([#36](#36)) ([138a847](138a847))
@github-actions
Copy link

github-actions bot commented May 2, 2023

πŸŽ‰ This PR is included in version 2.1.0-beta.3 πŸŽ‰

The release is available on:

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

github-actions bot pushed a commit that referenced this pull request May 2, 2023
* feat: get highlight data function

* feat: setup highlight testing

* feat: data interfaces

* refractor: split card codes

* feat: highlight cards code structure

* refractor: template shared folder

* refractor: card footer component

* refractor: separate o ut card style setup

* feat: highlights UI complete

* lint 138a847
@Deadreyo Deadreyo mentioned this pull request May 3, 2023
19 tasks
github-actions bot pushed a commit that referenced this pull request May 9, 2023
## [2.1.0](v2.0.1...v2.1.0) (2023-05-09)

### πŸ› Bug Fixes

* Overflowing language bar fix ([#38](#38)) ([3dc1a0a](3dc1a0a))

### πŸ• Features

* adding utilities for cards local generation & testing ([#34](#34)) ([c5a5fec](c5a5fec))
* Highlight Cards UI Generation (frontend-only) ([#36](#36)) ([138a847](138a847))
* highlights getting repo name & languages from pr's repo ([#41](#41)) ([1555a25](1555a25))
* higlight card upload & storage ([#39](#39)) ([b6abefa](b6abefa))
* UI adjustments ([#42](#42)) ([acbe81a](acbe81a))
@github-actions
Copy link

github-actions bot commented May 9, 2023

πŸŽ‰ This PR is included in version 2.1.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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feature: Highlight cards
3 participants