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

fix: improve loading performance of individual highlight #1349

Merged
merged 17 commits into from
Jul 14, 2023

Conversation

brandonroberts
Copy link
Contributor

@brandonroberts brandonroberts commented Jul 12, 2023

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

  • Adds direct link to OpenGraph image on individual highlight
  • Removes extra requests from SSR request for individual highlight
  • Fixes logic with selecting repo filters and opening individual highlights

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 Jul 12, 2023

βœ… Deploy Preview for oss-insights ready!

Name Link
πŸ”¨ Latest commit 77a6cf3
πŸ” Latest deploy log https://app.netlify.com/sites/oss-insights/deploys/64b192d3de45e400070edfcf
😎 Deploy Preview https://deploy-preview-1349--oss-insights.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.

@netlify
Copy link

netlify bot commented Jul 12, 2023

βœ… Deploy Preview for design-insights ready!

Name Link
πŸ”¨ Latest commit 77a6cf3
πŸ” Latest deploy log https://app.netlify.com/sites/design-insights/deploys/64b192d35a513e0008855a91
😎 Deploy Preview https://deploy-preview-1349--design-insights.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.

@brandonroberts brandonroberts marked this pull request as ready for review July 12, 2023 19:01
Copy link
Contributor

@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.

Issue Reference

In order to be considered for merging, the pull request description must refer to a specific issue number. This is described in our Contributing Guide.
This check is looking for a phrase similar to: "Fixes #XYZ" or "Resolves #XYZ" where XYZ is the issue number that this PR is meant to address.

@brandonroberts brandonroberts changed the title fix: increase loading performance of individual highlight fix: improve loading performance of individual highlight Jul 12, 2023
@bdougie
Copy link
Member

bdougie commented Jul 12, 2023

I had trouble QAing this. created a video

Rewatch.Screen.Recording.-.2023-07-12.at.2.37.06.PM.mp4

@brandonroberts
Copy link
Contributor Author

https://deploy-preview-1349--oss-insights.netlify.app

Interesting. This happens on current beta also without these changes, so I'll take into that also

Copy link
Member

@jpmcb jpmcb left a comment

Choose a reason for hiding this comment

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

Not sure if this is related, but the follow button's style doesn't look quite right to me on the beta deployment:

Screenshot 2023-07-13 at 10 52 05 AM

(notice how the follow button looks too wide and spills over to the left compared to the rest of the column)

On the live hightlights page, the alignment looks correct:

Screenshot 2023-07-13 at 10 52 26 AM

@brandonroberts
Copy link
Contributor Author

@bdougie @jpmcb should be good to go now.

  • Feed loads without issues
  • Individual Highlights pop up from clicking on one and fresh page reload
  • Repo filters are applied when clicked
  • Follow button in dropdown menu is aligned correctly

@brandonroberts brandonroberts requested a review from jpmcb July 14, 2023 18:23
Copy link
Member

@jpmcb jpmcb left a comment

Choose a reason for hiding this comment

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

Lookin good!!!!! πŸ•

@brandonroberts brandonroberts merged commit 9e193fe into beta Jul 14, 2023
@brandonroberts brandonroberts deleted the fix-highlights-opengraph branch July 14, 2023 19:13
github-actions bot pushed a commit that referenced this pull request Jul 14, 2023
## [1.56.0-beta.9](v1.56.0-beta.8...v1.56.0-beta.9) (2023-07-14)

### πŸ› Bug Fixes

* improve loading performance of individual highlight ([#1349](#1349)) ([9e193fe](9e193fe))
@github-actions
Copy link
Contributor

πŸŽ‰ This PR is included in version 1.56.0-beta.9 πŸŽ‰

The release is available on GitHub release

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

github-actions bot pushed a commit that referenced this pull request Jul 17, 2023
## [1.56.0](v1.55.0...v1.56.0) (2023-07-17)

### πŸ› Bug Fixes

* add hidden form for Netlify forms submission ([#1351](#1351)) ([8075012](8075012))
* ensure font color of selected interests pill changed is white  ([#1342](#1342)) ([b778f11](b778f11))
* improve loading performance of individual highlight ([#1349](#1349)) ([9e193fe](9e193fe))
* prevent concurrent user profile update requests ([#1372](#1372)) ([b23b9df](b23b9df))

### 🎨 Styles

* text inputs colors were not consistent in the user settings page ([#1360](#1360)) ([edae9c4](edae9c4))

### πŸ• Features

* add force login component to design system ([#1330](#1330)) ([9d68ffc](9d68ffc))
* add top contributors panel to feeds page ([#1347](#1347)) ([5b58dd5](5b58dd5))
* added new topics to the list ([#1357](#1357)) ([33f03bb](33f03bb))
* change position of visibility action component ([#1331](#1331)) ([f3f615c](f3f615c))
* changed the last commit date icon with a calender ([#1350](#1350)) ([2ae9d53](2ae9d53))
* implement featured highlights list ([#1337](#1337)) ([ed32594](ed32594))
* implement newsletter submission functionality ([#1332](#1332)) ([d996ac7](d996ac7))
* upgrade to Next 13.4, Supabase helper libs to 0.7.x ([#1301](#1301)) ([26876c2](26876c2))
@github-actions
Copy link
Contributor

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

The release is available on GitHub release

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.

4 participants