Skip to content

Conversation

@theianmay
Copy link
Member

Motivation

Build command did not include the image assets related to the Ratings components. After our direct integration of AirbnbRating (Swipe/Tap) from the third party library, we included image assets for the default ones that users have the ability to select. The below mentioned issue correctly pointed out that there was a flaw in our build process.

Fixes #3997

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

  • Yarn Build Test

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes generate no new warnings
  • New and existing unit tests pass locally with my changes

Additional context

After running yarn build, I verified that the image assets are present in the following location: ...\react-native-elements\node_modules@rneui\base\dist\AirbnbRating\images

@theianmay theianmay added this to the v5.0.0 milestone Nov 23, 2025
@theianmay theianmay self-assigned this Nov 23, 2025
@github-project-automation github-project-automation bot moved this to In Progress 🚧 in Community Roadmap 🛣 Nov 23, 2025
@codecov
Copy link

codecov bot commented Nov 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.29%. Comparing base (deb5834) to head (fbaae85).
⚠️ Report is 4 commits behind head on next.

Additional details and impacted files
@@           Coverage Diff           @@
##             next    #4004   +/-   ##
=======================================
  Coverage   79.29%   79.29%           
=======================================
  Files          90       90           
  Lines        2144     2144           
  Branches      957      960    +3     
=======================================
  Hits         1700     1700           
  Misses        442      442           
  Partials        2        2           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@theianmay theianmay merged commit 7c16f41 into react-native-elements:next Nov 23, 2025
9 checks passed
@github-project-automation github-project-automation bot moved this from In Progress 🚧 to Closed or Merged 🏁 in Community Roadmap 🛣 Nov 23, 2025
@theianmay theianmay deleted the fix/ratings-images branch November 23, 2025 06:24
github-actions bot pushed a commit that referenced this pull request Nov 23, 2025
fix: build process excluding image assets in published package
github-actions bot pushed a commit that referenced this pull request Nov 23, 2025
fix: build process excluding image assets in published package
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Closed or Merged 🏁

Development

Successfully merging this pull request may close these issues.

Missing assets in published package

1 participant