Skip to content

Conversation

rvsia
Copy link
Contributor

@rvsia rvsia commented Mar 17, 2020

Field wrapper is unnecessary layer => merged with render single field + refactoring for cleaner code

@rvsia rvsia added renderer React form renderer PR Technical debt V2 labels Mar 17, 2020
@rvsia rvsia requested a review from Hyperkid123 March 17, 2020 16:17
@rvsia rvsia changed the title [V2] refactor(renderer): refactor render single field [WIP ][V2] refactor(renderer): refactor render single field Mar 17, 2020
@codecov
Copy link

codecov bot commented Mar 17, 2020

Codecov Report

Merging #385 into v2 will decrease coverage by 0.02%.
The diff coverage is 60%.

Impacted file tree graph

@@            Coverage Diff            @@
##              v2     #385      +/-   ##
=========================================
- Coverage   87.8%   87.77%   -0.03%     
=========================================
  Files        133      132       -1     
  Lines       2198     2193       -5     
  Branches     699      699              
=========================================
- Hits        1930     1925       -5     
  Misses       268      268
Impacted Files Coverage Δ
...act-form-renderer/src/form-renderer/render-form.js 75% <60%> (-25%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 47b198b...e985ad8. Read the comment docs.

@rvsia rvsia force-pushed the refactorWrapper branch from 6d54413 to e985ad8 Compare March 17, 2020 16:41
@rvsia rvsia changed the title [WIP ][V2] refactor(renderer): refactor render single field [V2] refactor(renderer): refactor render single field Mar 17, 2020
@Hyperkid123 Hyperkid123 merged commit f128f78 into data-driven-forms:v2 Mar 18, 2020
@Hyperkid123
Copy link
Member

🎉 This PR is included in version 1.31.0 🎉

The release is available on

Demo can be found here!

@Hyperkid123
Copy link
Member

🎉 This PR is included in version 2.0.0 🎉

The release is available on

Demo can be found here!

@rvsia rvsia deleted the refactorWrapper branch April 28, 2020 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants