Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dependency Upgrades, Testing, and Documentation #19

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Haguili
Copy link
Collaborator

@Haguili Haguili commented Sep 27, 2023

Summary of Changes

  • Upgraded React version from 16.14.0 to 17.0.0
  • Upgraded Material-UI (MUI) from version 4 to version 5
  • Added some unit tests for various components
  • Added components documentation to provide better clarity and understanding of the codebase.
  • Updated the developer guide

This commit updates the start script in package.json to use the 'cross-env' package to set the 'HTTPS' environment variable. This change addresses the issue where running 'npm start' resulted in a 'cross-env' command not recognized error for windows.

- Updated the 'start' script in package.json
- Replaced 'HTTPS=true react-scripts start' with 'cross-env HTTPS=true react-scripts --openssl-legacy-provider start'
- Resolved the 'cross-env' command not recognized error for Windows
… improve code organization and maintainability. The changes include:

- Rearranging files into logical directories based on their functionality and purpose.
- Updating import paths and references to reflect the new directory structure.

These changes aim to enhance code clarity, separation of concerns, and ease of maintenance.
- Upgraded React version from 16.14.0 to 17.0.0
- Added unit tests for various components
- Added components documentation to provide better clarity and understanding of the codebase.
- Updated the developer guide
- Upgraded Material-UI (MUI) from version 4 to version 5 and 2 other dependancies
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.

None yet

1 participant