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/export pfb to workspace #681

Merged
merged 33 commits into from
Oct 4, 2020
Merged

Feat/export pfb to workspace #681

merged 33 commits into from
Oct 4, 2020

Conversation

ZakirG
Copy link
Contributor

@ZakirG ZakirG commented May 1, 2020

This PR adds a new button to the explorer in Windmill labeled "Export PFB to Workspace" that requests Pelican to place a cohort PFB in indexd and then sends that PFB GUID to the manifestservice.

Deployed in https://zakir.planx-pla.net/explorer

NOTE: The package.json will need to point to a new version of gen3-ui-component before merge. uc-cdis/gen3-ui-component#79

The change was called for by this ticket https://ctds-planx.atlassian.net/browse/PXP-5895

and specified by this design doc:
https://docs.google.com/document/d/1954SSzVgDDM41uBksIt9RY7_MRAcCzoaMfmwHtfpFG4/edit#

New Features

  • New button in Windmill, "Export PFB to Workspace", which requests Pelican to place a cohort PFB in indexd and then sends that PFB GUID to the manifestservice.

Deployment changes

  • A new Export to PFB button can be optionally enabled in the config.json. It is disabled by default. The button requires at least pelican:2020.10 release.

@lgtm-com
Copy link

lgtm-com bot commented May 1, 2020

This pull request introduces 2 alerts when merging ec45422 into b79f25b - view on LGTM.com

new alerts:

  • 2 for Potentially inconsistent state update

@lgtm-com
Copy link

lgtm-com bot commented Sep 14, 2020

This pull request introduces 2 alerts when merging 2ec29db into f341b1b - view on LGTM.com

new alerts:

  • 2 for Potentially inconsistent state update

@lgtm-com
Copy link

lgtm-com bot commented Sep 22, 2020

This pull request introduces 2 alerts when merging cd96315 into a6cea28 - view on LGTM.com

new alerts:

  • 2 for Potentially inconsistent state update

@lgtm-com
Copy link

lgtm-com bot commented Sep 23, 2020

This pull request introduces 2 alerts when merging da1e788 into a6cea28 - view on LGTM.com

new alerts:

  • 2 for Potentially inconsistent state update

@lgtm-com
Copy link

lgtm-com bot commented Sep 25, 2020

This pull request introduces 2 alerts when merging 51937c1 into a6cea28 - view on LGTM.com

new alerts:

  • 2 for Potentially inconsistent state update

@lgtm-com
Copy link

lgtm-com bot commented Sep 25, 2020

This pull request introduces 2 alerts when merging 40f7dc5 into a6cea28 - view on LGTM.com

new alerts:

  • 2 for Potentially inconsistent state update

@lgtm-com
Copy link

lgtm-com bot commented Sep 30, 2020

This pull request introduces 2 alerts when merging eb9d95e into cffd906 - view on LGTM.com

new alerts:

  • 2 for Potentially inconsistent state update

@lgtm-com
Copy link

lgtm-com bot commented Oct 1, 2020

This pull request introduces 2 alerts when merging 669c59d into cffd906 - view on LGTM.com

new alerts:

  • 2 for Potentially inconsistent state update

@lgtm-com
Copy link

lgtm-com bot commented Oct 1, 2020

This pull request introduces 2 alerts when merging 69ba709 into e0a3b1a - view on LGTM.com

new alerts:

  • 2 for Potentially inconsistent state update

@lgtm-com
Copy link

lgtm-com bot commented Oct 1, 2020

This pull request introduces 2 alerts when merging 417011b into e0a3b1a - view on LGTM.com

new alerts:

  • 2 for Potentially inconsistent state update

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.

2 participants