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

#80 Added JS Interpreter #226

Closed
wants to merge 3 commits into from

Conversation

THEBOSS0369
Copy link

I added ths js interpreter, now anyone learning will write the code in the textarea and the output will in the output section

Copy link

vercel bot commented Aug 8, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
learn-javascript ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 11, 2024 6:34am

@sumn2u
Copy link
Owner

sumn2u commented Aug 8, 2024

@THEBOSS0369 Thank you for creating this PR. Unfortunately, i couldn't see the changes on while previewing this PR. Can you guide me how to see the changes?

@THEBOSS0369
Copy link
Author

I forgot to add the super function it should be visible now

@sumn2u
Copy link
Owner

sumn2u commented Aug 8, 2024

I forgot to add the super function it should be visible now

Thanks for the prompt response. I'm still not seeing it. :(

@THEBOSS0369
Copy link
Author

Screenshot 2024-08-08 181518
Screenshot 2024-08-08 181705
See this i have added it should be visible

@sumn2u
Copy link
Owner

sumn2u commented Aug 8, 2024

Let me have a look into it.

@THEBOSS0369
Copy link
Author

Actually When i'm running server, i can't run the server after installing this project all dependencies and libraries as this project's gitbook and my node js have different version so they aren't working together. and when i tried to run it with docker the docker comes with many issues and i haven't used docker before. So i thought adding code and previewing from github might work

@sumn2u
Copy link
Owner

sumn2u commented Aug 8, 2024

Actually When i'm running server, i can't run the server after installing this project all dependencies and libraries as this project's gitbook and my node js have different version so they aren't working together. and when i tried to run it with docker the docker comes with many issues and i haven't used docker before. So i thought adding code and previewing from github might work

I tried with the code in this PR and it's not showing anything. I see one of the plugin that we can use but the problem is that plugin is outdated. Is it possible to create an issue of the system not running in your machine? I can have a look into it.

@THEBOSS0369
Copy link
Author

Yea Definitely, I will create it tomorrow. Also thanks for replying this fast before this I tried to contribute two more repositories but no one was replying and I'm a beginner so this conversation is very helpful for me growing as a programmer, Thanks Man it means Alot!

@THEBOSS0369
Copy link
Author

Hey @sumn2u I have add the issue

@THEBOSS0369
Copy link
Author

THEBOSS0369 commented Aug 11, 2024

Hey @sumn2u ! There is a problem occuring, i added js interpreter in every html file of the book directory and when the server is running it was showing on the website. But when i stopped the server and ran it again the html file went back to its original state. I have shared the screenshot in which i added the js interpreter but as soon i restart the server the code automatically get remove from the files.
working 1
working 2
These are the ss of the js interpreter i added. However, As soon i restart the server the interpreter will be gone.
When i searched this issue on google, then i came to know that , those file that are locked cannot be modified by contributor, i don't know how to resolve this.

@sumn2u
Copy link
Owner

sumn2u commented Aug 11, 2024

Hey @sumn2u ! There is a problem occuring, i added js interpreter in every html file of the book directory and when the server is running it was showing on the website. But when i stopped the server and ran it again the html file went back to its original state. I have shared the screenshot in which i added the js interpreter but as soon i restart the server the code automatically get remove from the files. working 1 working 2 These are the ss of the js interpreter i added. However, As soon i restart the server the interpreter will be gone. When i searched this issue on google, then i came to know that , those file that are locked cannot be modified by contributor, i don't know how to resolve this.

Hi @THEBOSS0369, we are using gitbook-plugin-theme-creative so any changes should be made there. I suggest creating a PR for that plugin and then using it as a dependency here. You may need to make some modifications, as it differs from the layout we have here.

@THEBOSS0369
Copy link
Author

Hi @sumn2u ! I have created pr for JS Interpreter in the gitbook plugin theme creative sumn2u/gitbook-plugin-theme-creative#1 . Please guide me what i have to do next ,so that the interpreter will show on the website.

@sumn2u
Copy link
Owner

sumn2u commented Aug 12, 2024

Hi @sumn2u ! I have created pr for JS Interpreter in the gitbook plugin theme creative sumn2u/gitbook-plugin-theme-creative#1 . Please guide me what i have to do next ,so that the interpreter will show on the website.

Nice work. Now, you can grab the commit SHA from that PR and add it as dependency in learn javascript.

Screenshot 2024-08-12 at 10 46 13 AM

Here the commit SHA starts with b1e336. I will copy it and add it in learn-javascript package.json.

 "devDependencies": {
    "@honkit/honkit-plugin-ga": "^1.0.1",
    "cz-conventional-changelog": "^3.3.0",
    "gitbook-plugin-chapter-fold": "^0.0.4",
    "gitbook-plugin-edit-link": "^2.0.2",
    "gitbook-plugin-exercises": "^3.0.0",
    "gitbook-plugin-favicon-custom": "^1.0.0",
    "gitbook-plugin-hide-published-with": "0.0.1",
    "gitbook-plugin-hints": "^1.0.2",
    "gitbook-plugin-intopic-toc": "^1.0.6",
    "gitbook-plugin-sharing": "https://github.com/sumn2u/plugin-sharing.git",
    "gitbook-plugin-sidebar-ad": "https://github.com/sumn2u/gitbook-plugin-sidebar-ad.git",
    "gitbook-plugin-sitemap": "^1.2.0",
    "gitbook-plugin-theme-creative": "https://github.com/sumn2u/gitbook-plugin-theme-creative.git#b1e336b9527493233bbd7e95a815b00987137bdd",
    "honkit": "^4.0.4",
    "honkit-plugin-i18nsettings": "^1.0.0"
  },

After this, run npm install to update the dependencies, then start the application and preview it. I can see the changes but it hided the sidebar and also the output is showing nothing.

screencapture-localhost-4000-en-2024-08-12-10_51_25

Screenshot 2024-08-12 at 10 52 43 AM

@THEBOSS0369
Copy link
Author

THEBOSS0369 commented Aug 13, 2024

Hey @sumn2u ! Check this message sumn2u/gitbook-plugin-theme-creative#1 and tell me whether the interpreter is still working or not. If it works, then i will proceed with the grabbing sha from the PR.

@sumn2u
Copy link
Owner

sumn2u commented Aug 13, 2024

Hey @sumn2u ! Check this message sumn2u/gitbook-plugin-theme-creative#1 and tell me whether the interpreter is still working or not. If it works, then i will proceed with the grabbing sha from the PR.

Thanks @THEBOSS0369. Your solution looks great! To cover most of the parts discussed here, I created PR #229. This PR utilizes a custom library that can be placed anywhere. Feel free to add any functionalities that are needed.

@sumn2u sumn2u closed this Aug 13, 2024
@THEBOSS0369
Copy link
Author

Welcome @sumn2u ! I have reviewed PR #229 and it looks good. I will soon start working on it and keep you updated with the progress. Also i have send you request on Linkedin My real name is Anuj

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

2 participants