Skip to content
This repository has been archived by the owner on Jan 20, 2022. It is now read-only.

AR-2129 Allow all form control children to be removed on subsequent renders #325

Merged

Conversation

justinanastos
Copy link
Contributor

@justinanastos justinanastos commented Mar 2, 2021

AR-2129

I should have done this with AR-2127, but I was lazy. This will cover all of the components and make a single test to capture all of them using test.each.

📦 Published PR as canary version: 8.17.2-canary.325.8623.0

✨ Test out this PR locally via:

npm install @apollo/space-kit@8.17.2-canary.325.8623.0
# or 
yarn add @apollo/space-kit@8.17.2-canary.325.8623.0

@justinanastos justinanastos added the bug fix Increment the patch version when merged label Mar 2, 2021
@justinanastos justinanastos force-pushed the justin/AR-2129/remove-all-form-controls-possible branch from c181d6f to dbe629d Compare March 2, 2021 18:57
@justinanastos justinanastos force-pushed the justin/AR-2129/remove-all-form-controls-possible branch from dbe629d to 5510fc4 Compare March 2, 2021 18:58
@justinanastos justinanastos requested a review from a team March 2, 2021 19:05
@justinanastos justinanastos marked this pull request as ready for review March 2, 2021 19:07
@justinanastos justinanastos changed the title Allow all form control children to be removed on subsequent renders AR-2129 Allow all form control children to be removed on subsequent renders Mar 2, 2021
@justinanastos justinanastos enabled auto-merge (squash) March 2, 2021 21:35
@justinanastos justinanastos merged commit 825c18a into main Mar 2, 2021
@justinanastos justinanastos deleted the justin/AR-2129/remove-all-form-controls-possible branch March 2, 2021 21:35
@apollo-bot2
Copy link
Collaborator

🚀 PR was released in v8.17.2 🚀

@apollo-bot2 apollo-bot2 added the released This issue/pull request has been released. label Mar 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug fix Increment the patch version when merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants