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

feat: Gitpod Support #389

Closed
wants to merge 14 commits into from
Closed

feat: Gitpod Support #389

wants to merge 14 commits into from

Conversation

sumeshir26
Copy link
Contributor

Make it easy for beginners to get started with contributing without installing Visual Studio, Node, and Windows Build Tools.

@Melvin-Abraham
Copy link
Owner

@sumeshir26 Instead of changing the initial configuration, we can instead try to figure if the application is running in a Gitpod environment (possibly via process.env). If it is running in Gitpod, then in that case we can set the value for the required config keys in assistantConfig right after we load the saved configuration.

@Melvin-Abraham
Copy link
Owner

@sumeshir26 Also, as of now I don't think this would be usable as the user can't upload their credentials or open the browser for authentication or have speaker input/output. Can you get that working somehow?

@sumeshir26
Copy link
Contributor Author

@Melvin-Abraham The Audio does not work, but if you upload the credentials to the Gitpod workspace, I will show up in the app. But the 'Get Tokens!' screen would not work because I cannot open a new browser window. A fix for that can be made if we display the link which is supposed to open in the app.
Attached is a video clip of the credentials working in Gitpod.
GAGitpod.zip

@sumeshir26
Copy link
Contributor Author

sumeshir26 commented May 18, 2021

Getting the link can be done by something like the PR you made for the google-assistant library and displaying a 'Copy Link' suggestion or display it on-screen. The PR can be pushed to your forked version of the library GAUDC now uses.

@Melvin-Abraham
Copy link
Owner

I'll make some changes to the documentation before merging :)

@Melvin-Abraham
Copy link
Owner

Getting the link can be done by something like the PR you made for the google-assistant library and displaying a 'Copy Link' suggestion or display it on-screen. The PR can be pushed to your forked version of the library GAUDC now uses.

@sumeshir26 Even though copying the auth link is implemented, the link isn't actually copied to the clipboard. Can you check if the whole workflow works for you?

@sumeshir26
Copy link
Contributor Author

sumeshir26 commented May 26, 2021

noVNC-CopyPaste.mp4

@Melvin-Abraham I actually found a solution for this. You can put the link on the main window so it can be copied. See attachment.

@sumeshir26
Copy link
Contributor Author

@Melvin-Abraham I need some help putting the link in the innerHtml of the noAuthGrid I looked on the web and tried to find the solution. I usually code in Python so this is kind of new to me. Thanks in advance!

@Melvin-Abraham
Copy link
Owner

@Melvin-Abraham I need some help putting the link in the innerHtml of the noAuthGrid I looked on the web and tried to find the solution. I usually code in Python so this is kind of new to me. Thanks in advance!

No issues. I will add that in... 👍

@obed9057
Copy link

obed9057 commented May 27, 2021 via email

@sumeshir26 sumeshir26 changed the title Gitpod Support feat: Gitpod Support May 27, 2021
@Melvin-Abraham
Copy link
Owner

@sumeshir26 Turns out copying the auth link to clipboard works as expected. The thing is the logic for "Copy Auth Link" and "Retry" were swapped due to which it seemed like copy didn't work. You can try it out by clicking on "Retry" instead.

I will push the commit for that soon. Meanwhile, the commit 2ff582d in this PR could be reverted.

@sumeshir26
Copy link
Contributor Author

@Melvin-Abraham Just reverted that...

@sumeshir26
Copy link
Contributor Author

@Melvin-Abraham This works perfectly now! And once noVNC gets this merged, the clipboard does not need to be used anymore.

@Melvin-Abraham
Copy link
Owner

I am halfway on the documentation part for Gitpod and will merge after its completion which will probably be after the next release

@im-coder-lg
Copy link
Contributor

@sumeshir26 I am using the Dockerfile code in another project since I don't know much on Linux packages. I noticed that the building takes a lot of time, so if you add this code to the .gitpod.yml file, it will generate prebuilds so that you can get started easily. I suggest adding this so that it may help people get started faster.
Code:

github:
  prebuilds:
    # enable for the master/default branch (defaults to true)
    master: true
# enable for all branches in this repo (defaults to false)
    branches: true
# enable for pull requests coming from this repo (defaults to true)
    pullRequests: true
# enable for pull requests coming from forks (defaults to false)
    pullRequestsFromForks: true
# add a "Review in Gitpod" button as a comment to pull requests (defaults to true)
    addComment: true
# add a "Review in Gitpod" button to pull requests (defaults to false)
    addBadge: true
# add a label once the prebuild is ready to pull requests (defaults to false)
    addLabel: prebuilt-in-gitpod

@im-coder-lg
Copy link
Contributor

Update: I don't think the Gitpod Prebuild/Review in Gitpod comments work on PRs, looks like Gitpod won't allow that unless it is allowed by them.

@sumeshir26
Copy link
Contributor Author

Closing due to inactivity

@sumeshir26 sumeshir26 closed this Jan 25, 2022
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.

4 participants