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: (Core) remove card-footer component #5158

Merged
merged 3 commits into from
May 3, 2021

Conversation

DeepakSap14
Copy link
Contributor

@DeepakSap14 DeepakSap14 commented Apr 19, 2021

Please provide a link to the associated issue.

fixes #5157
fixes #3929

BREAKING CHANGE:
CardFooter component has been removed.

Please provide a brief summary of this pull request.

Card should have only two areas.

  1. Card header
  2. Card content
    Card Footer needs to be removed.

Please check whether the PR fulfills the following requirements

Documentation checklist:

@DeepakSap14 DeepakSap14 added the bug Something isn't working label Apr 19, 2021
@DeepakSap14 DeepakSap14 added this to the Sprint 60 - Ariba milestone Apr 19, 2021
@DeepakSap14 DeepakSap14 self-assigned this Apr 19, 2021
@netlify
Copy link

netlify bot commented Apr 19, 2021

Deploy preview for fundamental-ngx ready!

Built with commit 6f13a66

https://deploy-preview-5158--fundamental-ngx.netlify.app

@DeepakSap14 DeepakSap14 requested review from InnaAtanasova, a team and pierrom April 19, 2021 12:29
@InnaAtanasova
Copy link
Contributor

Code looks good, let's see the Netlify preview and the build and will approve it.

@nikvalor nikvalor assigned nikvalor and unassigned nikvalor Apr 28, 2021
@DeepakSap14 DeepakSap14 added the core Core library specific issues label Apr 29, 2021
@InnaAtanasova
Copy link
Contributor

InnaAtanasova commented Apr 29, 2021

@DeepakSap14 when you introduce Breaking changes in a PR you need to provide info in the Wiki page of how the user should handle them. For example, you can introduce before and after code snippets to illustrate the migration. I did it for this PR but please add this in future:
Screen Shot 2021-04-29 at 11 45 26 AM

@DeepakSap14
Copy link
Contributor Author

@pierrom
can you please verify the issue #3929 using below link.
https://deploy-preview-5158--fundamental-ngx.netlify.app

@DeepakSap14
Copy link
Contributor Author

has been verified by @pierrom and issue is closed. merging this PR.

@DeepakSap14 DeepakSap14 merged commit 7a1c221 into main May 3, 2021
@DeepakSap14 DeepakSap14 deleted the 5157/fix-remove-card-footer branch May 3, 2021 07:39
N1XUS pushed a commit that referenced this pull request Jun 2, 2021
* fix:remove card-footer component

* removed card footer from api-file

* fix travis
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working core Core library specific issues
Projects
None yet
5 participants