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 json schema #2997

Merged

Conversation

richard-cox
Copy link
Contributor

@richard-cox richard-cox commented Sep 14, 2018

Updates to original PR - See #2664

  • Improved typing
  • Moved common elements into a new schema-form component
  • Allow users to toggle between form and text input whilst maintaining common value
  • Step validity bound to type of input (form valid vs valid json)
  • Updated form errors to be more material like
  • Fixed stepper navigation (would 're-enter' last step on pressing 'next' in last step
  • Fixed bleed of styles from angular6 form component (via fork)

hamzahamidi and others added 15 commits July 23, 2018 09:44
Signed-off-by: Hamza Hamidi <hamza.hamidi@orange.com>
Signed-off-by: Hamza Hamidi <hamza.hamidi@orange.com>
Signed-off-by: Hamza Hamidi <hamza.hamidi@orange.com>
Signed-off-by: Hamza Hamidi <hamza.hamidi@orange.com>
Signed-off-by: Hamza Hamidi <hamza.hamidi@orange.com>
Signed-off-by: Hamza Hamidi <hamza.hamidi@orange.com>
Signed-off-by: Hamza Hamidi <hamza.hamidi@orange.com>
Signed-off-by: Hamza Hamidi <hamza.hamidi@orange.com>
Signed-off-by: Hamza Hamidi <hamza.hamidi@orange.com>
Signed-off-by: Hamza Hamidi <hamza.hamidi@orange.com>
Signed-off-by: Hamza Hamidi <hamza.hamidi@orange.com>
Signed-off-by: Hamza Hamidi <hamza.hamidi@orange.com>
Signed-off-by: Hamza Hamidi <hamza.hamidi@orange.com>
- Improve typing
- Move common components into new `shema-form` component
- Allow users to toggle between form and text input whilst maintaining common value
- Step valid state bound to type of input (form valid vs valid json)
- Updated form errors to be more material like
- Fixed stepper navigation (would 're-enter' last step on pressing 'next' in last step
@cfdreddbot
Copy link

Hey richard-cox!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

@codecov
Copy link

codecov bot commented Sep 14, 2018

Codecov Report

Merging #2997 into v2-master will decrease coverage by 0.08%.
The diff coverage is 62.5%.

@@              Coverage Diff              @@
##           v2-master    #2997      +/-   ##
=============================================
- Coverage      71.02%   70.94%   -0.09%     
=============================================
  Files            607      608       +1     
  Lines          26250    26371     +121     
  Branches        5940     5970      +30     
=============================================
+ Hits           18645    18709      +64     
- Misses          7605     7662      +57

@richard-cox richard-cox changed the base branch from v2-master to feature-json-schema September 21, 2018 16:32
@richard-cox richard-cox merged commit 4c52565 into cloudfoundry:feature-json-schema Sep 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants