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

Lg web graph downloads #24

Merged
merged 6 commits into from
Mar 29, 2021
Merged

Lg web graph downloads #24

merged 6 commits into from
Mar 29, 2021

Conversation

james-strauss-uwa
Copy link
Collaborator

Allow users to download:

  • The Physical Graph Template after translation
  • The Physical Graph without deployment

I've added a new button beneath the translator PGT display ("Export to JSON")

Screen Shot 2020-10-15 at 2 14 23 pm

A click on the "Export to JSON" button calls client-side javascript, makeJSON(), which uses the existing /pgt_jsonbody server route to generate and return JSON.

Also added a checkbox near the 'Generate & Deploy Physical Graph' button ("Deploy")

Screen Shot 2020-10-15 at 2 14 43 pm

The "Deploy" checkbox is checked by default and the functionality remains the same. However, if the user un-checks the "Deploy" checkbox, the 'Generate & Deploy Physical Graph' button will change to read 'Generate Physical Graph', and when the user clicks on the button, the physical graph will be downloaded as JSON.

A small change to the /gen_pg route in lg web was required to support this functionality. It checks the form submission for a field named 'dlg_mgr_deploy'. If found, the deployment part of the route will be skipped and a JSON response is generated.

…' form submit button to reflect state of 'deploy' checkbox. Modified existing '/gen_pg' route on lg web to listen for 'deploy' form field and if false, return a JSON of the PG without deploying.
@coveralls
Copy link

coveralls commented Oct 15, 2020

Coverage Status

Coverage decreased (-0.1%) to 75.261% when pulling de592e1 on lg_web-graph-downloads into d8d63b3 on master.

@rtobar rtobar merged commit 947f71c into master Mar 29, 2021
@rtobar rtobar deleted the lg_web-graph-downloads branch July 7, 2021 07:23
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

3 participants