Add realtime (w/ fake data) trading table example - #6561
Open
riccardoperra wants to merge 33 commits into
Open
Conversation
…nts' into codex/realtime-trading-example
…into codex/realtime-trading-example
Angular,React,Solid improvements
Contributor
|
Important Review skippedToo many files! This PR contains 362 files, which is 62 over the limit of 300. To get a review, reduce the PR to 300 files or fewer by splitting it into smaller PRs or changing its base branch. Usage-priced reviews support at most 300 files. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (362)
You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
View your CI Pipeline Execution ↗ for commit 2b8624c
☁️ Nx Cloud last updated this comment at |
Contributor
🚀 Changeset Version Preview1 package(s) bumped directly, 0 bumped as dependents. 🟩 Patch bumps
|
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
- Replace `for...in` loops with `Object.keys()` for better performance and clarity. - Simplify the rendering logic by removing unnecessary checks and parameters. - Ensure consistent handling of injectors across component rendering.
…e-trading-example
riccardoperra
force-pushed
the
codex/realtime-trading-example
branch
from
August 18, 2026 15:58
f221f50 to
88d56df
Compare
…ing-example' into codex/realtime-trading-example
…table into codex/realtime-trading-example
KevinVandy
marked this pull request as ready for review
August 25, 2026 12:39
![nx-cloud[bot]](https://avatars.githubusercontent.com/in/80458?s=60&v=4){kind=small}
There was a problem hiding this comment.
Nx Cloud is proposing a fix for your failed CI:
We updated all realtime-trading example package.json files to align dependency versions with the rest of the workspace, fixing the 32 sherif version-consistency errors. Updating @glint/template from 1.7.10 to 1.8.0 in the Ember example resolves the TypeScript [Invoke] symbol incompatibility that caused the type-check failure, and correcting packageManager from pnpm@11.18.0 to pnpm@11.21.0 in the Angular example resolves the corepack version conflict that blocked the build.
Warning
- ❌ We could not verify this fix.
- The suggested diff is too large to display here, but you can view it on Nx Cloud ↗
Or Apply changes locally with:
npx nx-cloud apply-locally zjOt-cV9W
Apply fix locally with your editor ↗ View interactive diff ↗
🎓 Learn more about Self-Healing CI on nx.dev
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.