fugazi-io / webclient

web based terminal application for executing local and remote commands
http://fugazi.io
28 stars 12 forks source link

missing word in help command #84

Open chiptus opened 7 years ago

chiptus commented 7 years ago

image

I think this should be The client supports variables which can be assigned. I would love to fix that, just to get to know the codebase :)

nitzantomer commented 7 years ago

@chiptus Yeah, you're right, there's a missing be in there. Feel free to open a PR for fixing it, but I doubt that it will help you get to know the codebase, it's just a markdown file.

It can be found here: https://github.com/fugazi-io/webclient/blob/master/modules/markdown/help/help.md#variables (the module for the entire help part is this: https://github.com/fugazi-io/webclient/blob/master/modules/jsons/help.json)

Are you looking for an issue with more "meat" on it in order to delve into the code? If so, we can probably hook you up with something :)

chiptus commented 7 years ago

@nitzantomer of course it's easy :) anyway I'll fork it and see if I can get a local environment to work. And yes I would like to have something to fix

nitzantomer commented 7 years ago

@chiptus I've been told that the info about how to run the local environment doesn't work, and I still haven't had the time to get into it. BUT you can try to work with the next version using this branch: https://github.com/fugazi-io/webclient/tree/release/1.1.0 This version uses modules (import/export) and I know for a fact that the dev environment works out of the box, just do: npm run dev. More info about the changes in this version can be found here: https://github.com/fugazi-io/webclient/pull/83

(sorry for the mess)

As for something to fix, here are 3 open issues that you can look into:

They will all require you to get deep into the code.
Feel free to ask questions in gitter: https://gitter.im/fugazi-io/Lobby and I'll try to respond asap. Thanks!