Skip to content

Conformance documentation mm #1335

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

Open
wants to merge 3 commits into
base: alpha
Choose a base branch
from
Open

Conversation

MaryamMehd
Copy link
Collaborator

Issue #1325 - Conformance Document Updates (9f9eacf)
Updated Conformance.md based on comprehensive codebase analysis:
✅ URL item type: Changed from [ ] to [x] - Found complete UrlItem component with full validation
✅ Quantity item type: Changed from [ ] to [x] - Found complete QuantityItem component implementation
✅ minQuantity constraint: Changed from [ ] to [x] - Found complete validation and UI support
✅ maxQuantity constraint: Changed from [ ] to [x] - Found complete validation and UI support

- Mark url item type as fully implemented (was partial)
- Mark quantity item type as fully implemented (was partial)
- Mark minQuantity constraint as supported
- Mark maxQuantity constraint as supported

Based on codebase analysis showing complete implementations with dedicated components, validation, and UI support.

Closes #1325
✅ Added missing minQuantity and maxQuantity stories to SDC/BehaviorValueConstraints
✅ Combined redundant stories to save Chromatic quota:
  - Boolean: Combined BooleanBasic + BooleanBasicResponse → BooleanCombined
  - String: Combined StringBasic + StringBasicResponse → StringCombined
  - Integer: Combined IntegerBasic + IntegerBasicResponse → IntegerCombined
✅ Enhanced URL stories to show validation and multiple states
✅ Created new questionnaire assets with combined examples showing empty/filled states

This addresses conformance document updates where url and quantity are now fully implemented ([x])
and adds comprehensive Storybook coverage for minQuantity/maxQuantity constraints.
@@ -17,12 +17,12 @@ The list can be found here: http://hl7.org/fhir/R4/valueset-item-type.html
- [x] time
- [x] string
- [x] text
- [ ] url (partial implementation - using string UI component currently)
- [x] url
Copy link
Collaborator

Choose a reason for hiding this comment

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

We still don't have a "real" working URL - it's still a partial implementation using the string UI component

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