Skip to content

Update react and mui#30

Merged
JarnoRFB merged 3 commits intomainfrom
chore-AFC-6664
Mar 5, 2026
Merged

Update react and mui#30
JarnoRFB merged 3 commits intomainfrom
chore-AFC-6664

Conversation

@JarnoRFB
Copy link
Contributor

@JarnoRFB JarnoRFB commented Mar 5, 2026

Summary by CodeRabbit

  • Chores
    • Updated example client dependencies: React to v19, Material-UI to v7, and related type definitions.
    • Modified CI workflow configuration.

@vercel
Copy link

vercel bot commented Mar 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cash-register-api-example-client Ready Ready Preview, Comment Mar 5, 2026 10:15pm

Request Review

@coderabbitai
Copy link

coderabbitai bot commented Mar 5, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 1e4cd00e-2822-4ce3-9f79-2d476b10c146

📥 Commits

Reviewing files that changed from the base of the PR and between 8984900 and 31cda76.

⛔ Files ignored due to path filters (1)
  • example-client/pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (2)
  • .github/workflows/check-api-spec.yml
  • example-client/package.json

Walkthrough

This pull request makes two changes: removes the jobs: section from the GitHub Actions workflow file for API spec validation, which breaks the workflow structure since job definitions are no longer properly nested, and updates React-related dependencies in the example client package.json from React 18 to 19, Material-UI from 5 to 7, and corresponding type definitions.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes


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

@JarnoRFB JarnoRFB merged commit bf5ddc4 into main Mar 5, 2026
4 of 5 checks passed
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.

1 participant