Skip to content

CardView followup patch#2362

Merged
devongovett merged 6 commits into
mainfrom
cardview_followupv2
Sep 21, 2021
Merged

CardView followup patch#2362
devongovett merged 6 commits into
mainfrom
cardview_followupv2

Conversation

@devongovett
Copy link
Copy Markdown
Member

Based on #2271. Rebasing was too hard.

devongovett and others added 2 commits September 20, 2021 16:41
Co-authored-by: Danni <drobinson@livefyre.com>
Co-authored-by: Danni <drobinson@livefyre.com>
@adobe-bot
Copy link
Copy Markdown

Build successful! 🎉

@adobe-bot
Copy link
Copy Markdown

Build successful! 🎉

@adobe-bot
Copy link
Copy Markdown

Build successful! 🎉

Copy link
Copy Markdown
Member

@LFDanLu LFDanLu left a comment

Choose a reason for hiding this comment

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

LGTM, compared the diff of my local branch with this one and didn't find anything amiss. Pushed a small change to readd the ellipsis to the loadingMore strings

@adobe-bot
Copy link
Copy Markdown

Build successful! 🎉

@adobe-bot
Copy link
Copy Markdown

Build successful! 🎉

@LFDanLu LFDanLu mentioned this pull request Sep 21, 2021
5 tasks
@devongovett devongovett merged commit 1159686 into main Sep 21, 2021
@devongovett devongovett deleted the cardview_followupv2 branch September 21, 2021 22:15
paulkenney added a commit that referenced this pull request Sep 23, 2021
* main:
  Rename cards packages -> card
  Bump theme packages
  add rest of style props to action menu (#2374)
  Publish
  Add Autocomplete package (#2371)
  CardView followup patch (#2362)
  Creating chromatic baseline for fixing combobox to use useFormProps (#2334)
  Correcting Arabic translations of Loading and Loading more (#2359)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants