Skip to content
This repository has been archived by the owner on May 19, 2020. It is now read-only.

Don't persist errors in different service creation forms #1185

Merged
merged 1 commit into from
Aug 9, 2017

Conversation

el-mapache
Copy link
Contributor

@el-mapache el-mapache commented Aug 7, 2017

Maps service instance creation form errors to the underlying form object in the ServiceInstanceStore, not to the Form or CreateServiceInstance components. We clear the existing form each time a new service instance is selected for creation by calling the SERVICE_INSTANCE_CREATE_FORM_CANCEL action type

Handy GIF reference:
1-to-1-errors

removes goofy console logs

Some code cleanup + clear form properly on cancel

Fix lint error

Fixes broken specs

Add test for service plan component
@el-mapache el-mapache force-pushed the ab-1-1-instance-errors branch from 61473c7 to d987da4 Compare August 9, 2017 14:02
@jcscottiii jcscottiii merged commit 30bb761 into master Aug 9, 2017
@jcscottiii jcscottiii deleted the ab-1-1-instance-errors branch August 9, 2017 14:54
@el-mapache el-mapache changed the title Ab 1 1 instance errors Don't persist errors in different service creation forms Aug 9, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants