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

Cleaned up FieldDescription usage #2775

Merged
merged 4 commits into from
Apr 20, 2020

Conversation

Vultraz
Copy link
Contributor

@Vultraz Vultraz commented Apr 18, 2020

Small thing

@changeset-bot
Copy link

changeset-bot bot commented Apr 18, 2020

🦋 Changeset is good to go

Latest commit: 00c7c50

We got this.

This PR includes changesets to release 11 packages
Name Type
@arch-ui/fields Major
@keystonejs/fields-markdown Patch
@keystonejs/fields-mongoid Patch
@keystonejs/app-admin-ui Patch
@keystonejs/field-content Patch
@keystonejs/fields-wysiwyg-tinymce Patch
@keystonejs/fields Patch
@arch-ui/docs Patch
@keystonejs/cypress-project-basic Patch
@keystonejs/cypress-project-client-validation Patch
@keystonejs/cypress-project-social-login Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

@MadeByMike MadeByMike left a comment

Choose a reason for hiding this comment

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

I like this! Strangely this is actually a breaking change for arch-ui as the FieldDescription no longer accepts children.

Pretty sure nobody is using it directly but, just in case, can we please add a description of changes required to fix this in the changeset description.

@Vultraz
Copy link
Contributor Author

Vultraz commented Apr 20, 2020

Thanks! Sorry I didn't get to that yesterday.

@timleslie timleslie merged commit 78fda9d into keystonejs:master Apr 20, 2020
@Vultraz Vultraz deleted the field-description-cleanup branch April 20, 2020 23:56
This was referenced Apr 20, 2020
Wkasel pushed a commit to gosignal/keystone that referenced this pull request Apr 30, 2020
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