Skip to content

Fix demo userprofile - #1234

Merged
sBouzols merged 5 commits into
mainfrom
fix_demo_userprofile_2
Jul 9, 2026
Merged

Fix demo userprofile#1234
sBouzols merged 5 commits into
mainfrom
fix_demo_userprofile_2

Conversation

@sBouzols

@sBouzols sBouzols commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

PR Summary

sBouzols added 3 commits July 8, 2026 11:59
Signed-off-by: sBouzols <sylvain.bouzols@gmail.com>
Signed-off-by: sBouzols <sylvain.bouzols@gmail.com>
…emo app

Signed-off-by: sBouzols <sylvain.bouzols@gmail.com>
@coderabbitai

coderabbitai Bot commented Jul 8, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

@sBouzols, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 30 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 226513e6-952e-42ce-8161-d4ca5deffbe2

📥 Commits

Reviewing files that changed from the base of the PR and between 0db8843 and 5e04fe1.

📒 Files selected for processing (1)
  • demo/src/app.jsx
📝 Walkthrough

Walkthrough

The demo app replaces the user state with userProfile, deriving it from the auth event's e.user?.profile value. This new state is passed to TopBar and used to conditionally render authenticated tabs. The Connected demo UI section is also restructured with layout and component wiring changes.

Changes

userProfile State Migration and UI Restructuring

Layer / File(s) Summary
userProfile state, dispatch, and consumption
demo/src/app.jsx
AppContent initializes userProfile state instead of user, sets it from e.user?.profile ?? null in the auth dispatch handler, passes userProfile to TopBar, and conditionally renders tabs versus AuthenticationRouter based on userProfile !== null.
Connected demo layout restructuring
demo/src/app.jsx
The defaultTab "Connected" section is reorganized with hr separators using style={{ width: '100%' }}, new wrapper elements, and updated MultipleSelectionDialog wiring including isItemClickable logic and draggable list item styling.

Sequence Diagram(s)

sequenceDiagram
  participant AuthEvent
  participant AppContent
  participant TopBar
  participant AuthenticationRouter

  AuthEvent->>AppContent: USER event with e.user.profile
  AppContent->>AppContent: userProfile = e.user?.profile ?? null
  AppContent->>TopBar: userProfile prop
  alt userProfile !== null
    AppContent->>AppContent: render authenticated tabs
  else userProfile === null
    AppContent->>AuthenticationRouter: render router
  end
Loading
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description is just a template placeholder and does not describe the changes. Replace the placeholder with a brief summary of the demo userProfile and layout changes.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title matches the main demo user profile change, even if it is a bit broad.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@flomillot flomillot self-assigned this Jul 8, 2026
@sonarqubecloud

sonarqubecloud Bot commented Jul 9, 2026

Copy link
Copy Markdown

@sBouzols
sBouzols merged commit 24bc93d into main Jul 9, 2026
5 checks passed
@sBouzols
sBouzols deleted the fix_demo_userprofile_2 branch July 9, 2026 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants