Skip to content

Comments

Front end migration to only react#1076

Draft
SaboniAmine wants to merge 4 commits intomasterfrom
chore/migrate_pur_react
Draft

Front end migration to only react#1076
SaboniAmine wants to merge 4 commits intomasterfrom
chore/migrate_pur_react

Conversation

@SaboniAmine
Copy link
Member

Description

This PR introduces a refactoring of the front end app, with following changes :

  • Removing next.js app and replacing by a lightweight pure react version (build size went from 400 MB to 1.2 MB), with vite bundling
  • Factorizing all design tokens under the tailwind config
  • adding Zod runtime validation for API exchanges
  • remove periodic polling and replace by revalidate on focus or manual refresh with additional buttons
  • added accessibility labels

Related Issue

Please link to the issue this PR resolves: [issue #]

Motivation and Context

Why is this change required? What problem does it solve?

How Has This Been Tested?

Please describe in detail how you tested your changes.

Screenshots (if appropriate):

Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

Go over all the following points, and put an x in all the boxes that apply.

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING.md document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

feat(front): add modal to call deletion endpoints for projects

refactor: move dashboard logic to custom hook & helper

chore: fix date-range picker, remove console.log when not needed

chore: replace magic numbers by functional values,
standardise api call with fetchApi instead of fetch + injected url,
refactor Modal usage with custom hook

chore: standardize error handling

Test Python 3.14

feat(back): add cascade deletion of projects (#945)

feat(front): add modal to call deletion endpoints for projects

feat(back): add cascade deletion of projects
feat(front): add modal to call deletion endpoints for projects

chore: fix date-range picker, remove console.log when not needed

chore: finish cleaning

chore: lint
}
toast.error(error.message);
} else if (error instanceof ValidationError) {
console.error(`[ValidationError] ${error.endpoint}:`, error.zodError.issues);

Check failure

Code scanning / CodeQL

Use of externally-controlled format string High

Format string depends on a
user-provided value
.

Copilot Autofix

AI 1 day ago

Copilot could not generate an autofix suggestion

Copilot could not generate an autofix suggestion for this alert. Try pushing a new commit or if the problem persists contact support.

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