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

fix(body): update mandatory fields body #265

Merged
merged 3 commits into from
Mar 31, 2021
Merged

Conversation

WikiRik
Copy link
Member

@WikiRik WikiRik commented Mar 21, 2021

Why would you need a phone and address for bodies like NetCom and Les Anciens?

@WikiRik WikiRik requested a review from a team March 21, 2021 10:06
@codecov
Copy link

codecov bot commented Mar 21, 2021

Codecov Report

Merging #265 (d6388ae) into master (1cba493) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #265   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           42        42           
  Lines         1339      1339           
  Branches       211       211           
=========================================
  Hits          1339      1339           
Impacted Files Coverage Δ
models/Body.js 100.00% <ø> (ø)

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 7409e33...d6388ae. Read the comment docs.

@WikiRik WikiRik merged commit e80a5a7 into master Mar 31, 2021
@WikiRik WikiRik deleted the body-mandatory-fields branch March 31, 2021 11:05
serge1peshcoff pushed a commit that referenced this pull request Mar 31, 2021
## [1.34.1](1.34.0...1.34.1) (2021-03-31)

### Bug Fixes

* **body:** update mandatory fields body ([#265](#265)) ([e80a5a7](e80a5a7))
@serge1peshcoff
Copy link
Member

🎉 This PR is included in version 1.34.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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