Skip to content

fix(AirbnbRating): add props to parent component#3335

Merged
arpitBhalla merged 4 commits intoreact-native-elements:nextfrom
knmn2000:fix-airbnbRatings-props
Feb 4, 2022
Merged

fix(AirbnbRating): add props to parent component#3335
arpitBhalla merged 4 commits intoreact-native-elements:nextfrom
knmn2000:fix-airbnbRatings-props

Conversation

@knmn2000
Copy link
Copy Markdown
Contributor

@knmn2000 knmn2000 commented Feb 4, 2022

What kind of change does this PR introduce?
bugfix: Fixes the issue of props not being passed to module in the AirbnbRatings component

Did you add tests for your changes?
No. Please tell me how to write a test for this change, or if this change requires a test.
If relevant, did you update the documentation?
No, this change makes the component work in accordance with the documentation.
Summary
This change was made in response to issue #3289
with this change, the props are being passed correctly and the component works as expected.

Does this PR introduce a breaking change?
No.

Copy link
Copy Markdown
Member

@arpitBhalla arpitBhalla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to merge next first

@knmn2000
Copy link
Copy Markdown
Contributor Author

knmn2000 commented Feb 4, 2022

Merged into next @arpitBhalla, please review.

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 4, 2022

Codecov Report

Merging #3335 (e8efb9d) into next (09e1e31) will not change coverage.
The diff coverage is 50.00%.

❗ Current head e8efb9d differs from pull request most recent head 4448501. Consider uploading reports for the commit 4448501 to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##             next    #3335   +/-   ##
=======================================
  Coverage   79.96%   79.96%           
=======================================
  Files          88       88           
  Lines        1662     1662           
  Branches      731      731           
=======================================
  Hits         1329     1329           
  Misses        328      328           
  Partials        5        5           
Impacted Files Coverage Δ
packages/base/src/ListItem/ListItem.Accordion.tsx 7.40% <ø> (ø)
packages/base/src/AirbnbRating/AirbnbRating.tsx 66.66% <50.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 e614113...4448501. Read the comment docs.

@arpitBhalla arpitBhalla changed the title fix(AirbnbRatings): props not being passed to module fix(AirbnbRating): props not being passed to module Feb 4, 2022
@arpitBhalla arpitBhalla linked an issue Feb 4, 2022 that may be closed by this pull request
@arpitBhalla arpitBhalla changed the title fix(AirbnbRating): props not being passed to module fix(AirbnbRating): add props to parent component Feb 4, 2022
@arpitBhalla arpitBhalla merged commit f859d68 into react-native-elements:next Feb 4, 2022
github-actions bot pushed a commit that referenced this pull request Feb 4, 2022
Co-authored-by: Arpit Bhalla <arpitbhalla2001@gmail.com>
github-actions bot pushed a commit that referenced this pull request Feb 4, 2022
Co-authored-by: Arpit Bhalla <arpitbhalla2001@gmail.com>
github-actions bot pushed a commit that referenced this pull request Feb 4, 2022
Co-authored-by: Arpit Bhalla <arpitbhalla2001@gmail.com>
arpitBhalla added a commit that referenced this pull request Mar 17, 2022
Co-authored-by: Arpit Bhalla <arpitbhalla2001@gmail.com>
arpitBhalla added a commit that referenced this pull request Mar 19, 2022
Co-authored-by: Arpit Bhalla <arpitbhalla2001@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

AirbnbRating props are not sent to module in 4.0.0. beta 0

2 participants