-
Notifications
You must be signed in to change notification settings - Fork 620
[Dashboard] Replace Linkedin icon #4818
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
Conversation
|
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Your org has enabled the Graphite merge queue for merging into mainAdd the label “merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix. You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link. |
size-limit report 📦
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4818 +/- ##
=======================================
Coverage 47.67% 47.67%
=======================================
Files 1051 1051
Lines 56715 56715
Branches 3894 3894
=======================================
Hits 27040 27040
Misses 28988 28988
Partials 687 687
*This pull request uses carry forward flags. Click here to find out more. |
3c8b546 to
c8745ff
Compare
8abe611 to
80f4881
Compare
c8745ff to
814ce6c
Compare
80f4881 to
ba39ada
Compare
Merge activity
|
814ce6c to
60beea0
Compare
ba39ada to
5570651
Compare
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
This PR introduces a new `LinkedInIcon` component and replaces instances of the `SiLinkedin` icon with the new `LinkedInIcon` in two files, enhancing the consistency of icon usage across the application.
### Detailed summary
- Created a new `LinkedInIcon` component in `LinkedinIcon.tsx`.
- Replaced `SiLinkedin` with `LinkedInIcon` in `socialLinks.tsx`.
- Replaced `SiLinkedin` with `LinkedInIcon` in `PublisherSocials.tsx`.
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
60beea0 to
ffcd199
Compare
5570651 to
c31f919
Compare

Problem solved
Short description of the bug fixed or feature added
PR-Codex overview
This PR introduces a new
LinkedInIconcomponent and replaces the usage of theSiLinkedinicon fromreact-iconswith the new component in two files, enhancing the icon's integration and potentially its styling.Detailed summary
LinkedInIconcomponent inapps/dashboard/src/components/icons/brand-icons/LinkedinIcon.tsx.SiLinkedinwith<LinkedInIcon />in:apps/dashboard/src/components/footer/socialLinks.tsxapps/dashboard/src/components/contract-components/publisher/PublisherSocials.tsx