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

Feature/update shared components #142

Merged
merged 14 commits into from
Apr 25, 2023
Merged

Conversation

mr-nick17
Copy link
Contributor

What

Note this PR contains breaking changes, before merging and releasing a v1 branch will be created for any required maintenance

  • Breaking change moved breadcrumbs outside of <main> element; an accessibility requirement. Any consuming apps will need to remove the partial reference from their templates. Breadcrumbs should continue to render as currently after removal from consuming service.
  • Breaking change (I believe this only affects the dp-frontend-release-calendar). Updated existing <input> templates, consolidated model into one file (model.go) to avoid repetition and removed service specific model binding (a breaking change) and moved files to a common location.
  • New! created fields that consume inputs to adopt a standardised approach of implementation and field validation. Fields created:
    • Text <input>
    • <textarea> input
    • Radio <fieldset>
  • Updated readme with new fields and updated design system URLs

How to review

Sense check
Happy to demo any of the above ⬆️

Who can review

Frontend go dev

@mr-nick17 mr-nick17 force-pushed the feature/update-shared-components branch from 4b34d72 to 95966db Compare April 21, 2023 15:24
PATTERNS.md Outdated Show resolved Hide resolved
PATTERNS.md Outdated Show resolved Hide resolved
PATTERNS.md Outdated Show resolved Hide resolved
lindenmckenzie
lindenmckenzie previously approved these changes Apr 25, 2023
@mr-nick17 mr-nick17 merged commit d0b3897 into main Apr 25, 2023
@mr-nick17 mr-nick17 deleted the feature/update-shared-components branch April 25, 2023 10:25
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.

None yet

2 participants