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

feat: higlight card upload & storage #39

Merged
merged 6 commits into from
May 4, 2023
Merged

Conversation

nightknighto
Copy link
Contributor

@nightknighto nightknighto commented May 3, 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

Sets up the backend stage of highlight cards, i.e. uploading the generated image and returning it's link. Makes highlight cards ready for production.

  • An image is outdated after 1 day instead of 3 days, as the reactions in the card need to be updated more frequently,

Related Tickets & Documents

follow-up on #36
closes #40

Mobile & Desktop Screenshots/Recordings

https://opengraph-dev.sfo3.digitaloceanspaces.com/highlights/102.png
https://opengraph-dev.sfo3.digitaloceanspaces.com/highlights/103.png

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?

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 Passed!

@nightknighto nightknighto requested a review from 0-vortex May 3, 2023 09:24
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.

Cache change is required to get this to production, all other logic looks good to me! πŸ‘

@brandonroberts can you also give this a look? Other than the effort required to switch this off of GitHub rate limits to our public API, this is good to go live IMHO

src/social-card/highlight-card/highlight-card.service.ts Outdated Show resolved Hide resolved
@nightknighto
Copy link
Contributor Author

@0-vortex
/metadata route will now check last modified date against updated at date as well as saved reactions count against current reactios count

@nightknighto nightknighto requested a review from 0-vortex May 3, 2023 17:42
@nightknighto nightknighto merged commit b6abefa into beta May 4, 2023
@nightknighto nightknighto deleted the highlight-backend branch May 4, 2023 11:49
github-actions bot pushed a commit that referenced this pull request May 4, 2023
## [2.1.0-beta.4](v2.1.0-beta.3...v2.1.0-beta.4) (2023-05-04)

### πŸ• Features

* higlight card upload & storage ([#39](#39)) ([b6abefa](b6abefa))
github-actions bot pushed a commit that referenced this pull request May 4, 2023
* feat: higlight card upload & storage

* feat: added parse int pipe

* parseIntPipe

* type fixes

* feat: check if highlight needs regeneration

* lint b6abefa
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 subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
2 participants