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: add option to export xpress proposals #834

Merged
merged 10 commits into from
Nov 6, 2024
Merged

feat: add option to export xpress proposals #834

merged 10 commits into from
Nov 6, 2024

Conversation

deepaksftc
Copy link
Contributor

@deepaksftc deepaksftc commented Oct 29, 2024

Description

This PR introduces a feature that allows users to export Xpress proposals.

Motivation and Context

The need for this feature arose from the necessity to provide users with a way to export proposal data for offline usage and further analysis. This feature enhances the usability of the platform and provides additional value to our users.

Changes

  • Added an option to export Xpress proposals in the DownloadService.ts file.
  • Implemented the collection of proposal data for XLSX export in the proposal.ts file.
  • Created a new route in the xlsx.ts file to handle the export of the collected data.
  • Added e2e tests in the xpress.cy.ts file to ensure the feature works as expected.
  • Updated the frontend XpressProposalTable.tsx file to include the export option in the UI.

How Has This Been Tested

In addition to the unit tests that have been added, this feature has been manually tested to ensure its correct functionality. The e2e tests added in xpress.cy.ts confirm that an officer and a scientist can export proposals into Excel. The exported file was then manually inspected to ensure all data is present and correctly formatted.

How Has This Been Tested?

Fixes Issue

UserOfficeProject/issue-tracker#1028
UserOfficeProject/issue-tracker#1205

Depends On

Tests included/Docs Updated?

  • I have added tests to cover my changes.
  • All relevant doc has been updated

@deepaksftc deepaksftc marked this pull request as ready for review October 30, 2024 14:23
@deepaksftc deepaksftc requested a review from a team as a code owner October 30, 2024 14:23
@deepaksftc deepaksftc requested review from RasmiaKulan, mutambaraf, a team, martin-trajanovski and yoganandaness and removed request for a team and martin-trajanovski October 30, 2024 14:23
@simonfernandes simonfernandes enabled auto-merge (squash) November 6, 2024 09:19
@simonfernandes simonfernandes merged commit 78201f0 into develop Nov 6, 2024
19 checks passed
@simonfernandes simonfernandes deleted the 1028 branch November 6, 2024 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants