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

Command /jshell version result is broken visually. #1147

Closed
Alathreon opened this issue Aug 1, 2024 · 7 comments · Fixed by #1164
Closed

Command /jshell version result is broken visually. #1147

Alathreon opened this issue Aug 1, 2024 · 7 comments · Fixed by #1164
Assignees
Labels
bug Something isn't working enhance command Modify or improve an existing command or group of commands of the bot good first issue Good for newcomers

Comments

@Alathreon
Copy link
Contributor

Describe the bug

When using /jshell version, it results into something like this. The backsticks are not supposed to be visible.
image

Expected behavior

The backsticks shouldn't be here.

To Reproduce

Enter the command /jshell version.

Screenshots

Additional context

@Alathreon Alathreon added bug Something isn't working good first issue Good for newcomers enhance command Modify or improve an existing command or group of commands of the bot labels Aug 1, 2024
@andrew-09
Copy link
Contributor

andrew-09 commented Sep 15, 2024

Hi, I'd like to work on this, but when I run "/jshell version" I get this error because I don't have the jshell parameter set in config.json

image

What should I put for the jshell rest API URL?

image

@Alathreon
Copy link
Contributor Author

Hello, you need to setup https://github.com/Together-Java/JShellPlaygroundBackend locally in order to make it work locally.
And once the backend works, edit the base url by the url you are using for the backend.

@andrew-09
Copy link
Contributor

andrew-09 commented Sep 15, 2024

Hello, you need to setup https://github.com/Together-Java/JShellPlaygroundBackend locally in order to make it work locally. And once the backend works, edit the base url by the url you are using for the backend.

I think have both JShellAPI and JShellWrapper running but I am getting this error. I am using localhost:8080 as my rest URL

image

image

image

Logs:

image

image

@Alathreon
Copy link
Contributor Author

curl --request POST --url http://localhost:8080/jshell/single-eval --data '2+2'
Can you try to run this and tell me if it works ?
Also, I think it should be http://localhost:8080/jshell.

@andrew-09
Copy link
Contributor

curl --request POST --url http://localhost:8080/jshell/single-eval --data '2+2' Can you try to run this and tell me if it works ? Also, I think it should be http://localhost:8080/jshell.

Nope, didn't work:

image

Also changed to localhost:8080/jshell and it gave a new error

image

image

@Alathreon
Copy link
Contributor Author

Ah, maybe http://localhost:8080/jshell/ with a slash at the end, hopefully it will work.
Sorry, I don't have the exact url under my hand currently, if it doesn't work, I'll get it tomorrow.

@andrew-09
Copy link
Contributor

Thanks! The slash at the end fixed it

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhance command Modify or improve an existing command or group of commands of the bot good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants