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

SDT Frontend Implementation #209

Open
wants to merge 17 commits into
base: frontend-sdt
Choose a base branch
from

Conversation

Apurv428
Copy link
Collaborator

No description provided.

@Apurv428 Apurv428 marked this pull request as draft June 18, 2024 19:06
@deveshidwivedi
Copy link
Collaborator

Image is borderless on one of the sides, we could adjust it
Screenshot 2024-06-22 203932

@pleia2
Copy link
Contributor

pleia2 commented Jun 27, 2024

@Apurv428 Can you provide some precise instructions on how reviewers can test this change, including the dependencies? This will later be worked into the documentation.

@Apurv428
Copy link
Collaborator Author

Apurv428 commented Jun 27, 2024

To test the PR, please run the following commands:

  1. git clone https://github.com/Apurv428/software-discovery-tool.git
  2. cd software-discovery-tool
  3. git checkout sdt-frontend
  4. cd react-frontend
  5. npm i
  6. npm start

Note: You need node installed.

Alternatively, you can directly go to this link to view the current implementation: Link

@Apurv428
Copy link
Collaborator Author

Image is borderless on one of the sides, we could adjust it Screenshot 2024-06-22 203932

I'll make the necessary changes. Thanks for pointing it out.

Signed-off-by: Apurv Sonawane <[email protected]>
@Apurv428 Apurv428 force-pushed the sdt-frontend branch 5 times, most recently from 4ebf3fc to 6641078 Compare July 3, 2024 10:46
Signed-off-by: Apurv Sonawane <[email protected]>
Apurv428 and others added 6 commits July 3, 2024 16:18
Signed-off-by: Apurv Sonawane <[email protected]>
Signed-off-by: Apurv Sonawane <[email protected]>
Signed-off-by: Apurv428 <[email protected]>
Signed-off-by: Apurv Sonawane <[email protected]>
Signed-off-by: Apurv Sonawane <[email protected]>
Signed-off-by: Apurv Sonawane <[email protected]>
@Apurv428 Apurv428 force-pushed the sdt-frontend branch 2 times, most recently from 42bd7f1 to 1879b7b Compare July 3, 2024 10:50
@deveshidwivedi
Copy link
Collaborator

Hey @Apurv428, still getting errors on setting this up.

@Apurv428
Copy link
Collaborator Author

Apurv428 commented Jul 5, 2024

@deveshidwivedi Please share the error.

@deveshidwivedi
Copy link
Collaborator

@Apurv428 This is the one.
Commands I ran:
git clone https://github.com/Apurv428/software-discovery-tool.git
cd software-discovery-tool
git checkout sdt-frontend
npm i
npm start

@Apurv428
Copy link
Collaborator Author

Apurv428 commented Jul 5, 2024

@Apurv428 This is the one. Commands I ran: git clone https://github.com/Apurv428/software-discovery-tool.git cd software-discovery-tool git checkout sdt-frontend npm i npm start

can you share the error?

@deveshidwivedi
Copy link
Collaborator

Yes @Apurv428

 npm start

> [email protected] start
> honkit serve

(node:18188) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
(Use `node --trace-deprecation ...` to show where the warning was created)
Live reload server started on port: 35729
Press CTRL+C to quit ...

Starting server ...
info: parsing multilingual book, with 6 languages 
warn: no summary file in this book 
info: 19 plugins are installed 
info: 19 explicitly listed 
info: plugin "hints" is loaded
info: plugin "sidebar-ad" is loaded
info: plugin "intopic-toc" is loaded
info: plugin "sharing" is loaded
info: plugin "exercises" is loaded
info: plugin "@honkit/honkit-plugin-ga" is loaded
info: plugin "sitemap" is loaded
info: plugin "i18nsettings" is loaded
info: plugin "favicon-custom" is loaded
info: plugin "chapter-fold" is loaded
info: plugin "hide-published-with" is loaded
info: plugin "edit-link" is loaded
info: plugin "livereload" is loaded
info: plugin "highlight" is loaded
info: plugin "search" is loaded
info: plugin "lunr" is loaded
info: plugin "fontsettings" is loaded
info: plugin "theme-creative" is loaded
info: plugin "theme-default" is loaded
Error: EPERM: operation not permitted, lstat 'C:\Users\Win\AppData\Local\ElevatedDiagnostics'
    at Object.lstatSync [as statSync] (node:fs:1634:25)
    at C:\Users\Win\node_modules\honkit\lib\fs\node.js:17:39
    at C:\Users\Win\node_modules\immutable\dist\immutable.js:3016:46
    at List.__iterate (C:\Users\Win\node_modules\immutable\dist\immutable.js:2206:13)
    at mappedSequence.__iterateUncached (C:\Users\Win\node_modules\immutable\dist\immutable.js:3015:23)
    at seqIterate (C:\Users\Win\node_modules\immutable\dist\immutable.js:604:16)
    at IndexedSeq.__iterate (C:\Users\Win\node_modules\immutable\dist\immutable.js:320:14)
    at IndexedIterable.forEach (C:\Users\Win\node_modules\immutable\dist\immutable.js:4381:19)
    at C:\Users\Win\node_modules\immutable\dist\immutable.js:2069:14
    at Map.withMutations (C:\Users\Win\node_modules\immutable\dist\immutable.js:1353:7) {
  errno: -4048,
  code: 'EPERM',
  syscall: 'lstat',
  path: 'C:\\Users\\Win\\AppData\\Local\\ElevatedDiagnostics'
}

@Apurv428
Copy link
Collaborator Author

Apurv428 commented Jul 5, 2024

are you sure you are running in the right folder?
because it should come as [email protected] start

@deveshidwivedi
Copy link
Collaborator

are you sure you are running in the right folder? because it should come as [email protected] start

Oh right. It should be in the react-frontend folder. I can see the draft up now.
Could you please update the instructions to cd react-frontend after git checkout sdt-frontend?

@Apurv428 Apurv428 marked this pull request as ready for review July 9, 2024 11:23
@Apurv428 Apurv428 force-pushed the sdt-frontend branch 3 times, most recently from 132dbed to 1dc18dc Compare July 9, 2024 18:30
@Apurv428 Apurv428 changed the base branch from master to frontend-sdt July 11, 2024 12:47
Signed-off-by: Apurv Sonawane <[email protected]>
@Apurv428 Apurv428 force-pushed the sdt-frontend branch 6 times, most recently from f7d0e9d to eb9451d Compare July 11, 2024 13:59
Signed-off-by: Apurv Sonawane <[email protected]>
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.

5 participants