Skip to content
This repository was archived by the owner on Jun 11, 2024. It is now read-only.

modified handsontable, replace thrift client with new datahub client#205

Open
michellekhuang wants to merge 1 commit into
datahuborg:masterfrom
michellekhuang:master
Open

modified handsontable, replace thrift client with new datahub client#205
michellekhuang wants to merge 1 commit into
datahuborg:masterfrom
michellekhuang:master

Conversation

@michellekhuang

Copy link
Copy Markdown

Change the redirect url & client_id in main.js to debug on a different DataHub app.

@RogerTangos

Copy link
Copy Markdown
Contributor

Hey @michellekhuang, thanks for submitting the pull. @justinanderson and I are going to take a look at this tomorrow.

@RogerTangos

Copy link
Copy Markdown
Contributor

Hey @michellekhuang - How is this accessible to the end user. I see that SampleClient is added, but am really not sure how to use it.

@michellekhuang

Copy link
Copy Markdown
Author

@RogerTangos So this is run from a Datahub application, which still requires the user to change the redirect url. I asked Kimberly and she said this was the only way to get HandsOnTable to run

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants