-
Notifications
You must be signed in to change notification settings - Fork 1
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
Audit: How many components do not have storybook stories-new (#597) #622
base: develop
Are you sure you want to change the base?
Changes from all commits
74551bb
cdfe880
bb94ecb
43e89a4
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,230 @@ | ||
import '../styles.css'; | ||
|
||
# Audit of Storybook Component Documentation | ||
|
||
#### By Michael Larocca - github: [@MichaelLarocca](https://github.com/MichaelLarocca) | ||
|
||
--- | ||
|
||
## About | ||
|
||
#### **This documentation provides an overview of the audit process to identify components lacking comprehensive Storybook stories. The goal is to ensure all components are thoroughly documented and tested within Storybook, enhancing our codebase's overall quality and maintainability.** | ||
|
||
### Components | ||
|
||
<table> | ||
<thead> | ||
<tr> | ||
<th>Component Name</th> | ||
<th>Sufficient Story</th> | ||
<th>Insufficient Story</th> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What are we using to consider a story insufficient? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There are components with previous Stories written that must meet Shashi's standards. The 'Insufficient Storyfield' allows us to identify current and future Stories that must be rewritten to meet the required standards. (Per Shashi, we will be following the carbon design system for GIS Story standards) |
||
<th>Missing Story</th> | ||
</tr> | ||
</thead> | ||
<tbody> | ||
<tr> | ||
<td>AdminHeader</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>AdminLogo</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>AdminNav</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>AdminQuickMenu</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>AdminUserSettings</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>Alert</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>AlertDescription</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>AlertNotification</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>AlertTitle</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>Button</td> | ||
<td className="bg-green-100 text-center font-bold">X</td> | ||
<td></td> | ||
<td></td> | ||
</tr> | ||
<tr> | ||
<td>EliminatedStatus</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>EntryStatus</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>Form</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>GlobalSpinner</td> | ||
<td></td> | ||
<td className="bg-yellow-100 text-center font-bold">X</td> | ||
<td></td> | ||
</tr> | ||
<tr> | ||
<td>Heading</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>Input</td> | ||
<td className="bg-green-100 text-center font-bold">X</td> | ||
<td></td> | ||
<td></td> | ||
</tr> | ||
<tr> | ||
<td>Label</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>LeagueCard</td> | ||
<td className="bg-green-100 text-center font-bold">X</td> | ||
<td></td> | ||
<td></td> | ||
</tr> | ||
<tr> | ||
<td>LeagueEntries</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>LeagueSurvivors</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>LinkCustom</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>LoadingSpinner</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>Logo</td> | ||
<td className="bg-green-100 text-center font-bold">X</td> | ||
<td></td> | ||
<td></td> | ||
</tr> | ||
<tr> | ||
<td>LogoNav</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>Nav</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>NavDrawer</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>RadioGroup</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>RestPasswordForm</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>Table</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>TableColumns</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>TableData</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>TableDropDownMenu</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>UpdateEmailForm</td> | ||
<td></td> | ||
<td></td> | ||
<td className="bg-red-100 text-center font-bold">X</td> | ||
</tr> | ||
<tr> | ||
<td>WeeklyPickButton</td> | ||
<td className="bg-green-100 text-center font-bold">X</td> | ||
<td></td> | ||
<td></td> | ||
</tr> | ||
</tbody> | ||
</table> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please make this table using mdx syntax. To find out more, please look at table mdx syntax
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The current syntax is valid MDX for the table and is much easier to maintain, as we will continually add components. This syntax also makes Tailwind available to add background colors and additional formatting easily.
Shashi already approved my current version, but I will revise it if he changes his mind.