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

feat(ecosystem): add tier project count to feature cards #3477

Merged
merged 11 commits into from
Aug 4, 2023

Conversation

techtolentino
Copy link
Contributor

@techtolentino techtolentino commented Aug 1, 2023

Changes

Fixes #3472

Implementation details

  • adds function to getTierProjectCount
  • pass in count into the joinSectionCards to be used as a detail prop
  • added detail prop to feature card, and conditionally render if exists
  • styled overrides so the feature card is responsive

Preview link

https://qiskit-org-pr-3477.dcq4xc5i083.us-south.codeengine.appdomain.cloud/ecosystem

How to read this PR

Please review

  • method for getting project count (is this the best way?)
  • the new card.detail prop added
  • overriding styles added to ensure the Ecosystem page feature cards are responsive

Screenshots

Screen.Recording.2023-08-01.at.10.58.15.AM.mov

Copy link
Contributor

@y4izus y4izus left a comment

Choose a reason for hiding this comment

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

just 2 small things Randy!

components/Ui/UiCompactFeature.vue Outdated Show resolved Hide resolved
components/Ui/UiCompactFeature.vue Outdated Show resolved Hide resolved
Co-authored-by: Yaiza <yaiza.garcia.mm@gmail.com>
Copy link
Collaborator

@eddybrando eddybrando left a comment

Choose a reason for hiding this comment

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

Thank you very much @techtolentino, it looks very good!

I left a couple of comments.

pages/ecosystem.vue Outdated Show resolved Hide resolved
pages/ecosystem.vue Outdated Show resolved Hide resolved
pages/ecosystem.vue Outdated Show resolved Hide resolved
components/Ui/UiJoinWithCards.vue Outdated Show resolved Hide resolved
components/Ui/UiCompactFeature.vue Outdated Show resolved Hide resolved
Copy link
Collaborator

@eddybrando eddybrando left a comment

Choose a reason for hiding this comment

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

Nice job @techtolentino! Thanks a lot! 💪

@y4izus y4izus merged commit 71e92a0 into main Aug 4, 2023
5 checks passed
@y4izus y4izus deleted the feat/ecosystem-add-tier-project-count branch August 4, 2023 14:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Ecosystem] add project count to Leadspace tier cards
3 participants