Skip to content

[DSM-179] Implement CSV Exports UI - #79

Open
goldsmithb wants to merge 18 commits into
mainfrom
feature/csv-exports-p2-rebase
Open

goldsmithb wants to merge 18 commits into
mainfrom
feature/csv-exports-p2-rebase

Conversation

@goldsmithb

Copy link
Copy Markdown
Contributor

Jira Ticket


This PR is for the UI Pages associated with CSV Exports.

/csv_exports - Export Summaries Table

  • A server-paginated table that displays the CSV exports associated with the user (shows all of them if admin)

/csv_exports/:id - Export Details Page

  • A page for viewing the details of a csv export
  • renders the selection (export paths) as a client-paginated table with page size of 50

Tests

Route tests are included for the above new pages in app/javascript/src/app/routes/__tests__/

Other changes and things to note

  • Renamed 'objects' to 'files' and 'directories' to 'folders'
  • Added two sign in options for development environment: 'admin' and 'user'
  • Pass omniauth sign-in data to react app through custom root element data attributes --- allows us to create dynamic sign-in & out links based on the environment -- we render a form for sign-out just like we would in a rails view
  • create utilities to grab and set CSRF token headers for API requests that require it to be included (non-read actions, usually)
  • Created a navbar in the React app with links to the s3 browser and csv export table
  • Moved sign out link from rails view to react navbar
  • Moved the Notifications component to a root layout that renders within the router context -- allows us to use react router links within notifications (in this PR, a link to view a csv export that was just ordered).
  • Added support for server-side pagination to the generic table builder component
  • renamed the ObjectDetailField component to just DetailField because it is reused in csv export details view
  • I included code that can use a perPage URL search param when getting server-paginated data, but it currently is ignored. If we want to allow this in the future it is there (but I could also remove it if we think it's unnecessary).
  • reset selection store state when an export is successfully requested
  • removed code that supported displaying entire bucket selection (not allowed)

@goldsmithb
goldsmithb marked this pull request as ready for review September 1, 2026 21:17
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.

1 participant