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

Remove contents list component #915

Merged
merged 1 commit into from
Jun 1, 2018
Merged

Conversation

andysellick
Copy link
Contributor

@andysellick andysellick commented May 30, 2018

The contents list component has been moved to the gem. This PR updates government-frontend to use the component there, and remove it from here.

  • update contents list with body component and other references to use contents list component from gem
  • update tests

Trello card: https://trello.com/c/aDQ8nEKO/90-modify-component-contents-list

Visual regression results:
https://government-frontend-pr-915.surge.sh/gallery.html

Component guide for this PR:
https://government-frontend-pr-915.herokuapp.com/component-guide

@tijmenb tijmenb temporarily deployed to government-frontend-pr-915 May 30, 2018 13:29 Inactive
@andysellick andysellick changed the title Remove contents list component [DO NOT MERGE] Remove contents list component May 30, 2018
@andysellick andysellick requested a review from vanitabarrett May 31, 2018 10:20
@andysellick andysellick force-pushed the remove-contents-list-component branch from a4e5d85 to 25f5f1f Compare June 1, 2018 09:36
@boffbowsh boffbowsh temporarily deployed to government-frontend-pr-915 June 1, 2018 09:36 Inactive
@boffbowsh boffbowsh temporarily deployed to government-frontend-pr-915 June 1, 2018 09:40 Inactive
@andysellick andysellick changed the title [DO NOT MERGE] Remove contents list component Remove contents list component Jun 1, 2018
@boffbowsh boffbowsh temporarily deployed to government-frontend-pr-915 June 1, 2018 10:44 Inactive
- update contents list with body component and other references to use contents list component from gem
- update tests
@andysellick andysellick merged commit 26b4998 into master Jun 1, 2018
@andysellick andysellick deleted the remove-contents-list-component branch June 1, 2018 12:26
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.

3 participants