Skip to content

Container component#92

Merged
mwarman merged 8 commits intomainfrom
91-container-component
Mar 11, 2025
Merged

Container component#92
mwarman merged 8 commits intomainfrom
91-container-component

Conversation

@mwarman
Copy link
Member

@mwarman mwarman commented Mar 11, 2025

📢 Instructions

  • Begin with a DRAFT pull request.
  • Follow italicized instructions to add detail to assist the reviewers.
  • After completing all checklist items, change the pull request to READY.

🔧 Change Summary

Describe the changes included in this pull request. Link to the associated GitHub issue(s).

  • fixes Container component #91
  • Added the Container component.
  • Created a components sub-page for the Container component.
  • Moved the Page component to the /common/content source directory.
  • Created a components sub-page for the Page component.

📝 Checklist

Pull request authors must complete the following tasks before marking the PR as ready to review.

  • Complete a self-review of changes
  • Unit tests have been created or updated
  • The code is free of [new] lint errors and warnings
  • Update storybook stories as needed
  • Update project documentation as needed, README, JSDoc, etc.

🧪 Steps to Test

Describe the process to test the changes in this pull request.

View the Container component in Storybook. In the app...

  1. Regression test. The container component did not alter the UX, but rather encapsulated it into a standard component.

🔗 Additional Information

Optionally, provide additional details, screenshots, or URLs that may assist the reviewer.

  • [...]

@mwarman mwarman self-assigned this Mar 11, 2025
@mwarman mwarman linked an issue Mar 11, 2025 that may be closed by this pull request
@mwarman mwarman marked this pull request as ready for review March 11, 2025 15:33
@mwarman mwarman merged commit 7f0419c into main Mar 11, 2025
5 checks passed
@mwarman mwarman deleted the 91-container-component branch March 11, 2025 15:33
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.

Container component

1 participant