Skip to content

Conversation

@MarkRedeman
Copy link
Contributor

πŸ“ Description

  • Update to react 19 so that it is consistent with the other Geti apps.

✨ Changes

Select what type of change your PR is:

  • πŸš€ New feature (non-breaking change which adds functionality)
  • 🐞 Bug fix (non-breaking change which fixes an issue)
  • πŸ”„ Refactor (non-breaking change which refactors the code base)
  • ⚑ Performance improvements
  • 🎨 Style changes (code style/formatting)
  • πŸ§ͺ Tests (adding/modifying tests)
  • πŸ“š Documentation update
  • πŸ“¦ Build system changes
  • 🚧 CI/CD configuration
  • πŸ”§ Chore (general maintenance)
  • πŸ”’ Security update
  • πŸ’₯ Breaking change (fix or feature that would cause existing functionality to not work as expected)

βœ… Checklist

Before you submit your pull request, please make sure you have completed the following steps:

  • πŸ“š I have made the necessary updates to the documentation (if applicable).
  • πŸ§ͺ I have written tests that support my changes and prove that my fix is effective or my feature works (if applicable).
  • 🏷️ My PR title follows conventional commit format.

For more information about code review checklists, see the Code Review Checklist.

@MarkRedeman MarkRedeman requested a review from maxxgx September 10, 2025 12:42
@MarkRedeman MarkRedeman changed the title πŸ”§ Update Geti Inspect to react 19 πŸ”§ chore(geti-inspect)Update Geti Inspect to react 19 Sep 10, 2025
@MarkRedeman MarkRedeman changed the title πŸ”§ chore(geti-inspect)Update Geti Inspect to react 19 πŸ”§ chore(geti-inspect) Update Geti Inspect to react 19 Sep 10, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the Geti Inspect application to use React 19 for consistency with other Geti applications. The changes encompass dependency upgrades, configuration adjustments, and backend improvements.

  • Upgraded React and related dependencies from version 18 to 19
  • Updated development tooling and added new React compiler plugins
  • Corrected backend configuration and file paths

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

File Description
app/ui/package.json Updated React dependencies to v19, added React compiler plugins, corrected package name, and refined engine requirements
app/ui/README.md Added syntax highlighting to code block
app/backend/src/main.py Added FastAPI configuration options and updated default port
app/backend/run.sh Corrected file paths from app/ to src/ directory

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@MarkRedeman MarkRedeman changed the title πŸ”§ chore(geti-inspect) Update Geti Inspect to react 19 πŸ”§ chore(geti-inspect): Update Geti Inspect to react 19 Sep 10, 2025
@MarkRedeman MarkRedeman merged commit 8e2a5e4 into open-edge-platform:feature/geti-inspect Sep 10, 2025
14 of 21 checks passed
MarkRedeman added a commit to MarkRedeman/anomalib that referenced this pull request Sep 23, 2025
…tform#2948)

* Use src folder inside run.sh

* Set openapi_url path

* Update to react 19

* Specify bash language in readme
MarkRedeman added a commit that referenced this pull request Oct 8, 2025
* Use src folder inside run.sh

* Set openapi_url path

* Update to react 19

* Specify bash language in readme
camiloHimura pushed a commit to camiloHimura/anomalib that referenced this pull request Nov 11, 2025
…tform#2948)

* Use src folder inside run.sh

* Set openapi_url path

* Update to react 19

* Specify bash language in readme
maxxgx pushed a commit that referenced this pull request Nov 26, 2025
* Use src folder inside run.sh

* Set openapi_url path

* Update to react 19

* Specify bash language in readme
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.

3 participants