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

Pc report csv #63

Merged
merged 7 commits into from
Aug 9, 2023
Merged

Pc report csv #63

merged 7 commits into from
Aug 9, 2023

Conversation

pconrad
Copy link
Contributor

@pconrad pconrad commented Aug 7, 2023

In this PR, we add the ability for an admin to download a CSV copy of the report lines associated with a given report.

Screenshots

What's new is the Download CSV button seen in this screenshot:

image

When you click it, you get a CSV file such as this one, shown here in the "Numbers" program on MacOS:

image

To test

  1. Login as an admin
  2. Create a commons
  3. Join the commons
  4. Buy a cow or two
  5. Go to the Admin / Jobs page and run the instructor report
  6. Go to the Admin/ Instructor Reports Page
  7. Click View Report
  8. Click Download CSV
  9. Compare the contents of the CSV with the report; they should match.

@pconrad pconrad force-pushed the pc-report-csv branch 2 times, most recently from 80b20ca to 8a8e15d Compare August 8, 2023 01:55
… in pom.xml

The excluded method is in pom.xml in a <excludedMethods> parameter
@vChase42
Copy link
Collaborator

vChase42 commented Aug 9, 2023

LGTM

@pconrad pconrad merged commit 66bb6cb into main Aug 9, 2023
14 checks passed
pconrad added a commit that referenced this pull request Aug 31, 2023
pconrad added a commit that referenced this pull request Jun 7, 2024
Feature: Endpoints for Suspending and Restoring Users
pconrad added a commit that referenced this pull request Jun 7, 2024
PR: FIX: BUG [nonexistent commons returns blank page / commons chat should not be available to users who are not part of a commons]
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