RAJAGOPALAN-GANGADHARAN / JS-OS

An Unified Operating System on the web
59 stars 27 forks source link

Update package.json #45

Closed ArchismanKarmakar closed 2 years ago

ArchismanKarmakar commented 2 years ago

Issue link

Removed Packages :

Unused dependencies
* @babel/runtime
* @fortawesome/fontawesome-svg-core
* @fortawesome/free-solid-svg-icons
* @fortawesome/react-fontawesome
* @monaco-editor/react
* @react-pdf/renderer
* argparse
* glot-api
* html-to-react
* i
* jquery
* monaco-editor
* npm
* pdf-viewer-reactjs
* react-ace
* react-calendar
* react-fontawesome
* react-froala-wysiwyg
* react-items-carousel
* react-pdf
* react-read-pdf
* react-responsive-carousel
*** react-router
* react-scripts
* react-slideshow-image
* react-spreadsheet
* react-trumbowyg
* react-viewer
Unused devDependencies
* gh-pages
* react-app-rewired
Missing dependencies
* regenerator-runtime: .\src\index.js
* ace: .\src\defaultapps\texteditor\src\ext-emmet.js
* @tinymce/tinymce-react: .\src\defaultapps\markdown\markdown_editor.js
* prop-types: .\src\defaultapps\blogfeed\components\CodeBlock.js
RAJAGOPALAN-GANGADHARAN commented 2 years ago

@ArchismanKarmakar , Thank you for this PR. Are you sure it doesn't break any changes? I will also test this locally once - this weekend(pardon my delay, work related things)

ArchismanKarmakar commented 2 years ago

@ArchismanKarmakar , Thank you for this PR. Are you sure it doesn't break any changes? I will also test this locally once - this weekend(pardon my delay, work related things)

Yes pretty bit. You can also try it. You can use DPCHECK

From the beginning only I was not able to start the server by npm start it showed some error, but the error was due to a package called three, three was throwing error so I couldn't compile & check.

It's okay when you're busy everything goes out of mind, we can't do anything except just following our routine. - please don't mind for this, irrespective of my age I am telling it you sorry sir.

ArchismanKarmakar commented 2 years ago

@ArchismanKarmakar , Thank you for this PR. Are you sure it doesn't break any changes? I will also test this locally once - this weekend(pardon my delay, work related things)

i am not able to run it because the packages are outdated and npm i is giving error at react now. Can you please provide your node_modules folder please

ArchismanKarmakar commented 2 years ago

I fixed everything now it's running in my machine also. few node_modules were causing error. updating package & package lock now

ArchismanKarmakar commented 2 years ago

i kept few unused packages because after removing them npm is throwing error

RAJAGOPALAN-GANGADHARAN commented 2 years ago

@ArchismanKarmakar Thanks for the update, can you please remove package-lock.json from this commit? Ive been meaning to remove that from repository, Once deleted could you please add that to gitignore as well

ArchismanKarmakar commented 2 years ago

@ArchismanKarmakar Thanks for the update, can you please remove package-lock.json from this commit? Ive been meaning to remove that from repository, Once deleted could you please add that to gitignore as well

check now sir

ArchismanKarmakar commented 2 years ago

@ArchismanKarmakar Thanks for the update, can you please remove package-lock.json from this commit? Ive been meaning to remove that from repository, Once deleted could you please add that to gitignore as well

27 issue solved

RAJAGOPALAN-GANGADHARAN commented 2 years ago

Moved to #48