Fix/dropdown input result callbacks #1071
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Deploying js-sdk-toolchain with
|
| Latest commit: |
0055a23
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://a59d0414.js-sdk-toolchain.pages.dev |
| Branch Preview URL: | https://fix-dropdown-input-result-ca.js-sdk-toolchain.pages.dev |
Test this pull request
|
gonpombo8
force-pushed
the
fix/dropdown-input-result-callbacks
branch
from
4c32990 to
974cff6
Compare
pravusjif
approved these changes
Feb 27, 2025
|
This fixes decentraland/unity-explorer#3272 |
gonpombo8
force-pushed
the
fix/dropdown-input-result-callbacks
branch
from
b16c3e6 to
447ab3e
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1071 +/- ##
==========================================
+ Coverage 62.04% 71.63% +9.58%
==========================================
Files 599 159 -440
Lines 19986 5310 -14676
Branches 2413 963 -1450
==========================================
- Hits 12401 3804 -8597
+ Misses 7163 1373 -5790
+ Partials 422 133 -289 ☔ View full report in Codecov by Sentry. |
gonpombo8
added a commit
that referenced
this pull request
Mar 27, 2025
* chore: update readmes (#1054) * chore: update readmes * Update packages/@dcl/ecs/README.md Co-authored-by: Gon Pombo <[email protected]> Signed-off-by: Juan Cazala <[email protected]> --------- Signed-off-by: Juan Cazala <[email protected]> Co-authored-by: Gon Pombo <[email protected]> * add filters for asset explorer (#1056) * add filters for asset explorer * use decentraland/oddish-action * move 'Filter' to enum * move IAsset type determination to fn * fix several issues with import files (#1059) * fix race condition when validating models on import * add 'VALUE_NOT_IN_RANGE' to ignored model errors * fix import button on asset explorer styles * move feeded fs example model to Models/example/model.glb * reset the FileInput value so the same file can be selected again * fix import file size + extension validation (#1060) * change max file import size to 50mb * allow all extensions on FileInput to show errors on Error modal * fix: Load smart item asset path (#1062) * fix: Load smart items path * fix: Add a small retry when fetching the models to avoid race conditions * fix: Linter * fix: handle new categories without breaking (#1058) * fix Animator component initialization (#1061) * fix network entities for the editor (#1069) * fix network entities for the editor * fix tests & lint * fix tests * feat: Make createReactBasedUiSystem public (#1063) * feat: Make createReactBasedUiSystem public * fix: playground-assets.api.md * feat: Send PlayersHelpers to the initAssetPacks method * feat: Update BasicView Actions to handle TextArea fields (#1064) * Fix/dropdown input result callbacks (#1071) * fix uiDropdown not being updated * fix tests * fix test COVERAGE * remove unnecessary return * wip test * move 'spawn' ts checker to 'fork' (#1074) * fix several issues (#1072) * fix scroll on assets catalog * fix tooltip link on Action & Trigger components * deprecate alphaTexture on PBR & add it to unlit materials * replace 'ctrl' for '⌘' on shortcuts when necessary * add 'global' prop to AudioSource * feat: avatarTarget param addition in movePlayerTo() (#1073) * feat: Add AdminTools Smart Item (#1066) * WIP * feat: Export components Accordion and CheckboxGroup * fix: Update type of AddButton * feat: Add AdminTools Rewards Control Inspector * feat: Add AdminTools Video Control Inspector * feat: Add AdminTools Smart Items Control Inspector * feat: Remove unused code * chore: Use dev @dcl/asset-packs package * chore: Update dependencies * wip * wip * fix: Config * chore: Update @dcl/asset-packs dependency * feat: Update Pill padding styles * feat: Update VideoControl styles * feat: Move TextAnnouncementControl section to a new component * feat: Update SmartItemControl styles * feat: Update RewardsControl styles * feat: Move AdminAllowListControl section to a new component * feat: Refactor AdminToolkitView * chore: Update @dcl/asset-packs version * feat: Add border styles to the Accordion component * fix: AdminTools styles * refactor: RewardInspector component form to a new component * feat: Add RewardsBasicView component for Collectible Dispenser * feat: Add AdminToolsBasicView for AdminTools * fix: lint * fix: Rewards component visibility * feat: Add InfoTooltip for the AdminTools view * eat: Add InfoTooltip for the Rewards view * fix: Typo in Admin Text announcements control * fix: Prioritize Video Control over Text Announcements * fix: Rewards URL env * chore: Update @dcl/asset-packs version * chore: Update @dcl/asset-packs version * fix: Lint * feat: Add parentEntity, removeParent, getParent, getChildren methods to the initAssetPacks init * fix: Update asset-packs * fix: Bundle * fix: Update asset-packs * fix: Update asset-packs * fix: Update asset-packs * fix: Update asset-packs * fix: Remove link all screens and show author checkboxes * fix: Remove link all smart items handler * fix: Remove Airdrops section * fix: Show action Claim Airdrop only when the entity has the Rewards component * fix: Update asset-packs * fix: Update asset-packs * fix: Doesn't allow add rows * fix: AvailableActions Map * fox: Remove validation for dev * update asset-packs --------- Co-authored-by: Gonzalo DCL <[email protected]> * bump asset-packs 2.2.1 (#1077) * update left/right click button behaviour on renderer (#1078) * update left/right click button behaviour on renderer * remove gizmo right click button * fixed review comments * remove desktop client (OLD) option (#1083) * select "Scene" in tree on startup (#1082) * update left/right click button behaviour on renderer * select 'Scene' in tree on startup * add refresh button for Assets (#1085) * support border in uitransforms (#1084) * support border in uitransforms * fix build * change protocol to experimental * use main protocol * fix snapshots test * fix tests * fix coverage * update protocol --------- Signed-off-by: Juan Cazala <[email protected]> Co-authored-by: Juan Cazala <[email protected]> Co-authored-by: Nicolas Echezarreta <[email protected]> Co-authored-by: Gabriel Díaz <[email protected]> Co-authored-by: Pravus <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.