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

Export BMP Spreadsheet Tool to HydroShare #2899

Merged
merged 1 commit into from
Aug 3, 2018

Conversation

rajadain
Copy link
Member

@rajadain rajadain commented Aug 2, 2018

Overview

The MMW BMP Spreadsheet Tool has a number of spreadsheets that allow modification and manipulation of BMP data. Stroud hopes to use it in places where MMW doesn't have in-built functionality. By exporting it with the rest of the files to HydroShare, we ensure that users always have immediate access to it, no matter where they download the data from. #2898 added it to the MMW UI. This adds it to they HydroShare export.

The MMW BMP Spreadsheet Tool is fetched from the latest version on GitHub, so it is always the most recent. It will only be added on the first export, not every update afterwards.

Connects #2894

Demo

https://beta.hydroshare.org/resource/c7637c31f3bf4496b6851c020764e0d4/

image

Testing Instructions

  • Populate the MMW_HYDROSHARE_SECRET_KEY in the app and worker VMs using the "HydroShare Beta" item in LastPass
  • Go to https://beta.hydroshare.org/ and setup an account if you don't already have one
  • Check out this branch and go to :8000/
  • Go to "My Account" → "Linked Accounts" and link to HydroShare (it should automatically connect to the beta site)
  • Make a new MapShed project.
  • Export to HydroShare. Ensure the export completes successfully.
  • Ensure the exported HydroShare Resource has the Spreadsheet Tool on it. Download it from HydroShare and verify.

When exporting MapShed projects, we now also include the
MMW BMP Spreadsheet Tool. This is fetched for the latest
version on GitHub, so it is always the most recent. This
will only be added on the first export, not every update
afterwards.
@caseycesari
Copy link
Member

Code looks straightforward, just waiting on a sign-up verification email from beta.hydroshare.org.

Copy link
Member

@caseycesari caseycesari left a comment

Choose a reason for hiding this comment

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

@caseycesari caseycesari assigned rajadain and unassigned caseycesari Aug 3, 2018
@rajadain
Copy link
Member Author

rajadain commented Aug 3, 2018

Thanks for reviewing!

@rajadain rajadain merged commit aebaf8c into develop Aug 3, 2018
@rajadain rajadain deleted the tt/hydroshare-include-bmp-spreadsheet-tool branch August 3, 2018 19:43
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