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

(PXP-6540): Feat/explorer download formats #780

Merged
merged 10 commits into from
Jan 19, 2021

Conversation

cterrazas2
Copy link
Contributor

@cterrazas2 cterrazas2 commented Jan 6, 2021

See Guppy for remainder of PXP-6540 changes.

New Features

  • Added support for TSV and CSV downloads from Explorer.

Deployment Changes

  • The new feature of Portal requires Guppy v0.10.0
  • Added new button types in the Portal config. See portal_config.md for an example.

@mfshao
Copy link
Collaborator

mfshao commented Jan 6, 2021

can you deploy this to somewhere we can test?

Copy link
Collaborator

@mfshao mfshao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add margin-right: 10px; to the stylesheet that is styling explorer-button-group__dropdown to make the button into the right position?

Also you need to update your local package.json to use the guppy from your GH branch and commit in the updated lock file. Otherwise the portal will not work properly in your dev env because it is not using the new Guppy component. For now if I click on TSV button in your dev env I can only get an error msg in console

@mfshao
Copy link
Collaborator

mfshao commented Jan 19, 2021

Suggest to add a brief sentence to show that we added a few new explorer button types to the PR description as Deployment Changes

Copy link
Collaborator

@mfshao mfshao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

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.

3 participants