Skip to content
This repository has been archived by the owner on May 24, 2022. It is now read-only.

Consolidate CustomerLogo components #29

Closed
katjuell opened this issue Mar 9, 2022 · 2 comments
Closed

Consolidate CustomerLogo components #29

katjuell opened this issue Mar 9, 2022 · 2 comments
Labels
team/content-platform Content Platform Team related tickets.

Comments

@katjuell
Copy link
Contributor

katjuell commented Mar 9, 2022

We have a static and animated version of our CustomerLogos Component. We should consolidate the data so both components use the same data.

Differences between the two components

Logos in static CustomerLogosSection version not in the animated version:

'Button',
'Expanse',
'GetYourGuide',
'Outreach',
'Thorn',

Logos in animated CustomerLogosSectionAnimated version not in the static version:

'Blend',
'Cambridge University Press',
'Capella Space',
'Cornerstone OnDemand',
'Dropbox',
'Eventbrite',
'Factset',
'Handy',
'Mercado Libre',
'Mercari',
'Motley Fool',
'Reddit',
'Skelter Labs',
'TripActions',
'Unity',
'Workiva',

Let's merge the 5 left behind logos from the static list in the animated list.

@katjuell katjuell added the team/content-platform Content Platform Team related tickets. label Mar 9, 2022
@katjuell katjuell moved this to Backlog in Content Platform Work Mar 9, 2022
@katjuell katjuell moved this from Backlog to Ready for Development in Content Platform Work Mar 9, 2022
@bretthayes bretthayes self-assigned this Mar 22, 2022
@bretthayes bretthayes moved this from Ready for Development to In Progress in Content Platform Work Mar 22, 2022
@bretthayes bretthayes removed their assignment Mar 22, 2022
@bretthayes bretthayes moved this from In Progress to Blocked/Paused in Content Platform Work Mar 22, 2022
@bretthayes
Copy link
Contributor

Moving this to blocked because we have Homepage updates underway which may affect the rewrite of both of these components. This is also related to #35.

@bretthayes
Copy link
Contributor

Closing this in lieu of the new static CustomerLogos component which will be ported over when I do #40.

Repository owner moved this from Blocked/Paused ⏳ to Done ✅ in Content Platform Work Apr 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
team/content-platform Content Platform Team related tickets.
Projects
No open projects
Status: Done
Development

No branches or pull requests

2 participants